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 endpoint /internal/evaluations/runs/search to search evaluation runs by ID or variant name

Important

Add /internal/evaluations/runs/search endpoint to search evaluation runs by ID or variant name, with backend and frontend support.

  • Backend:
    • Adds search_evaluation_runs_handler in search_runs.rs to handle /internal/evaluations/runs/search endpoint.
    • Implements search_evaluation_runs in evaluation_queries.rs to query evaluation runs by ID or variant name.
    • Adds EvaluationRunSearchResult struct in evaluation_queries.rs for deserializing search results.
  • Frontend:
    • Updates EvalRunSelector.tsx to use SearchEvaluationRunResult for displaying search results.
    • Modifies route.ts to call searchEvaluationRuns from TensorZeroClient.
    • Updates EvaluationRunBadge.tsx to use SearchEvaluationRunResult.
  • Models:
    • Adds SearchEvaluationRunResult and SearchEvaluationRunsResponse types in SearchEvaluationRunResult.ts and SearchEvaluationRunsResponse.ts.
  • Tests:
    • Adds tests for search_evaluation_runs in evaluation_queries.rs and search_runs.rs.

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

@shuyangli shuyangli force-pushed the sl/migrate-search-evaluation-runs branch from b05ed9a to b70af49 Compare December 12, 2025 00:15
@shuyangli shuyangli force-pushed the sl/migrate-count-datapoints-for-evaluations branch 2 times, most recently from 51397f3 to c77307a Compare December 12, 2025 15:24
@shuyangli shuyangli force-pushed the sl/migrate-count-datapoints-for-evaluations branch 5 times, most recently from 5b494bf to 2d415e1 Compare December 16, 2025 19:01
@shuyangli shuyangli force-pushed the sl/migrate-search-evaluation-runs branch from b70af49 to bd8d235 Compare December 16, 2025 19:51
@shuyangli shuyangli force-pushed the sl/migrate-search-evaluation-runs branch from bd8d235 to be07d05 Compare December 16, 2025 22:06
Base automatically changed from sl/migrate-count-datapoints-for-evaluations to main December 17, 2025 03:43
@shuyangli shuyangli changed the title [Stacked] Migrate searchEvaluationRuns Migrate searchEvaluationRuns Dec 17, 2025
@shuyangli shuyangli force-pushed the sl/migrate-search-evaluation-runs branch from be07d05 to eda13aa Compare December 17, 2025 03:54
@shuyangli shuyangli marked this pull request as ready for review December 17, 2025 03:56
@shuyangli shuyangli requested a review from virajmehta December 17, 2025 03:56
@graphite-app
Copy link

graphite-app bot commented Dec 17, 2025

Merge activity

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

virajmehta
virajmehta previously approved these changes Dec 17, 2025
@virajmehta virajmehta assigned shuyangli and unassigned virajmehta Dec 17, 2025
Copy link
Member Author

Conflicts fixed

@shuyangli shuyangli assigned virajmehta and unassigned shuyangli Dec 17, 2025
@shuyangli shuyangli added this pull request to the merge queue Dec 17, 2025
@virajmehta virajmehta assigned shuyangli and unassigned virajmehta Dec 17, 2025
@Aaron1011 Aaron1011 removed this pull request from the merge queue due to a manual request Dec 17, 2025
@Aaron1011 Aaron1011 added this pull request to the merge queue Dec 17, 2025
Merged via the queue into main with commit 84b08c9 Dec 17, 2025
55 checks passed
@Aaron1011 Aaron1011 deleted the sl/migrate-search-evaluation-runs branch December 17, 2025 17:33
@shuyangli shuyangli mentioned this pull request Dec 18, 2025
52 tasks
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.

4 participants