Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@shuyangli
Copy link
Member

@shuyangli shuyangli commented Dec 10, 2025

Added an API endpoint /internal/evaluations/run-stats to count evaluation runs and migrated the UI to use this endpoint instead of direct ClickHouse queries.


Important

Added /internal/evaluations/run-stats endpoint to count evaluation runs, replacing direct ClickHouse queries in the UI.

  • API Endpoint:
    • Added /internal/evaluations/run-stats in internal.rs to count evaluation runs.
    • Implemented handler get_evaluation_run_stats_handler in count_runs.rs.
  • Backend:
    • Added count_total_evaluation_runs() in evaluation_queries.rs for ClickHouse query.
    • Defined EvaluationQueries trait in evaluation_queries.rs.
    • Added EvaluationRunStatsResponse type in types.rs.
  • UI:
    • Replaced countTotalEvaluationRuns with getTensorZeroClient().countEvaluationRuns() in route.tsx and index.tsx.
    • Removed countTotalEvaluationRuns from evaluations.server.ts.
  • Tests:
    • Added E2E test test_count_evaluation_runs in evaluations.rs.

This description was created by Ellipsis for 663b806. You can customize this summary. It will automatically update as commits are pushed.

@shuyangli shuyangli mentioned this pull request Dec 10, 2025
52 tasks
@shuyangli shuyangli force-pushed the sl/port-count-total-evaluation-runs branch 2 times, most recently from 1991457 to 814948b Compare December 11, 2025 00:11
@shuyangli shuyangli force-pushed the sl/port-count-total-evaluation-runs branch from 814948b to 9373839 Compare December 11, 2025 04:47
@shuyangli shuyangli force-pushed the sl/port-count-total-evaluation-runs branch from 9373839 to 316d0f7 Compare December 11, 2025 05:11
@shuyangli shuyangli marked this pull request as ready for review December 11, 2025 05:11
@shuyangli shuyangli force-pushed the sl/port-count-total-evaluation-runs branch from 316d0f7 to 293f1f8 Compare December 11, 2025 05:18
@shuyangli shuyangli force-pushed the sl/port-count-total-evaluation-runs branch from 293f1f8 to ab5c79e Compare December 11, 2025 14:56
@shuyangli shuyangli requested review from virajmehta and removed request for GabrielBianconi December 11, 2025 14:56
virajmehta
virajmehta previously approved these changes Dec 11, 2025
@virajmehta virajmehta assigned shuyangli and unassigned virajmehta Dec 11, 2025
@shuyangli shuyangli added this pull request to the merge queue Dec 11, 2025
@shuyangli shuyangli removed this pull request from the merge queue due to a manual request Dec 11, 2025
@Aaron1011 Aaron1011 enabled auto-merge December 11, 2025 19:40
@Aaron1011 Aaron1011 added this pull request to the merge queue Dec 11, 2025
@Aaron1011 Aaron1011 removed this pull request from the merge queue due to a manual request Dec 11, 2025
@Aaron1011 Aaron1011 added this pull request to the merge queue Dec 11, 2025
@Aaron1011 Aaron1011 removed this pull request from the merge queue due to a manual request Dec 11, 2025
@Aaron1011 Aaron1011 added this pull request to the merge queue Dec 11, 2025
@Aaron1011 Aaron1011 removed this pull request from the merge queue due to a manual request Dec 11, 2025
@Aaron1011 Aaron1011 added this pull request to the merge queue Dec 11, 2025
Merged via the queue into main with commit a09aa99 Dec 11, 2025
55 checks passed
@Aaron1011 Aaron1011 deleted the sl/port-count-total-evaluation-runs branch December 11, 2025 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants