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

Skip to content

Conversation

@shuyangli
Copy link
Member

@shuyangli shuyangli commented Dec 11, 2025

Added a new API endpoint /internal/evaluations/runs that returns a paginated list of evaluation runs


Important

Adds a new API endpoint /internal/evaluations/runs to list evaluation runs with pagination, including backend query handling and frontend integration.

  • API Endpoint:
    • Adds /internal/evaluations/runs endpoint in internal.rs for listing evaluation runs.
    • Implements list_evaluation_runs_handler in list_runs.rs to handle requests.
  • Database Queries:
    • Implements list_evaluation_runs in evaluation_queries.rs to query ClickHouse for evaluation runs with pagination.
    • Adds EvaluationRunInfoRow struct in evaluation_queries.rs for deserialization.
  • TypeScript Bindings:
    • Adds EvaluationRunInfo.ts and ListEvaluationRunsResponse.ts for TypeScript type definitions.
    • Updates index.ts to export new types.
  • Frontend Integration:
    • Updates EvaluationRunsTable.tsx and route.tsx to use new API for fetching evaluation runs.
    • Removes getEvaluationRunInfo from evaluations.server.ts and related tests in evaluations.test.ts.
  • Testing:
    • Adds tests for list_evaluation_runs in evaluation_queries.rs and list_runs.rs to ensure correct functionality.

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

@shuyangli shuyangli force-pushed the sl/migrate-list-evaluation-runs branch 2 times, most recently from e233393 to afab906 Compare December 11, 2025 16:40
@shuyangli shuyangli mentioned this pull request Dec 11, 2025
52 tasks
@shuyangli shuyangli force-pushed the sl/port-count-total-evaluation-runs branch from ab5c79e to 663b806 Compare December 11, 2025 19:35
@shuyangli shuyangli force-pushed the sl/migrate-list-evaluation-runs branch from afab906 to 004b1ab Compare December 11, 2025 19:39
Base automatically changed from sl/port-count-total-evaluation-runs to main December 11, 2025 22:48
@shuyangli shuyangli force-pushed the sl/migrate-list-evaluation-runs branch from 004b1ab to 28ec67c Compare December 12, 2025 00:15
@shuyangli shuyangli changed the title [Stacked] Migrate list evaluation runs to gateway Migrate list evaluation runs to gateway Dec 12, 2025
@shuyangli shuyangli force-pushed the sl/migrate-list-evaluation-runs branch from 28ec67c to 46df08a Compare December 12, 2025 15:22
@shuyangli shuyangli marked this pull request as ready for review December 12, 2025 15:24
@graphite-app
Copy link

graphite-app bot commented Dec 12, 2025

Merge activity

  • Dec 12, 4:39 PM UTC: This pull request can not be added to the Graphite merge queue. Please try rebasing and resubmitting to merge when ready.
  • Dec 12, 4:39 PM UTC: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..

@shuyangli shuyangli added this pull request to the merge queue Dec 12, 2025
Merged via the queue into main with commit cfae918 Dec 12, 2025
55 checks passed
@shuyangli shuyangli deleted the sl/migrate-list-evaluation-runs branch December 12, 2025 18:35
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.

3 participants