-
Notifications
You must be signed in to change notification settings - Fork 763
Migrate get/countWorkflowEvaluationRunEpisodesByTaskName #5287
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6c30994 to
544ffff
Compare
6b9c70e to
6e976dd
Compare
Member
Author
|
/merge-queue |
Contributor
|
🚀 Merge queue workflow triggered! View the run: https://github.com/tensorzero/tensorzero/actions/runs/20380888929 |
6e976dd to
dbb1834
Compare
…valuationRunEpisodesByTaskName to backend This moves the workflow evaluation episodes by task name queries from the UI (direct ClickHouse calls) to the backend (Rust API endpoints). Backend changes: - Add GroupedWorkflowEvaluationRunEpisodeWithFeedbackRow type to workflow_evaluation_queries.rs - Add list_workflow_evaluation_run_episodes_by_task_name and count_workflow_evaluation_run_episode_groups_by_task_name trait methods - Implement ClickHouse queries with complex CTEs for grouping by task_name - Add unit tests using MockWorkflowEvaluationQueries - Add e2e database tests in tests/e2e/db/workflow_evaluation_queries.rs - Add route handlers at /internal/workflow-evaluations/episodes-by-task-name and /internal/workflow-evaluations/episodes-by-task-name/count - Add e2e endpoint tests (require gateway restart) Frontend changes: - Generate and export TypeScript bindings for new types - Add listWorkflowEvaluationRunEpisodesByTaskName and countWorkflowEvaluationRunEpisodeGroupsByTaskName methods to TensorZeroClient - Update route.tsx to use new client methods instead of direct ClickHouse calls - Delete unused getWorkflowEvaluationRunEpisodesByTaskName and countWorkflowEvaluationRunEpisodesByTaskName functions from workflow_evaluations.server.ts - Delete GroupedWorkflowEvaluationRunEpisodeWithFeedback Zod schema and type - Remove corresponding test for deleted function
dbb1834 to
f045d22
Compare
GabrielBianconi
approved these changes
Dec 20, 2025
Member
|
python install timed out last time |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This moves the workflow evaluation episodes by task name queries from the UI (direct ClickHouse calls) to the backend (Rust API endpoints).
Backend changes:
Frontend changes:
Important
Migrate workflow evaluation episode queries from frontend to backend, adding new Rust API endpoints and updating frontend to use these endpoints.
GroupedWorkflowEvaluationRunEpisodeWithFeedbackRowtype inworkflow_evaluation_queries.rs.list_workflow_evaluation_run_episodes_by_task_nameandcount_workflow_evaluation_run_episodes_by_task_nameinworkflow_evaluation_queries.rs.internal.rsfor/internal/workflow-evaluations/episodes-by-task-nameand/internal/workflow-evaluations/episodes-by-task-name/count.workflow_evaluation_queries.rsand e2e tests intests/e2e/db/workflow_evaluation_queries.rs.TensorZeroClientfor listing and counting episodes by task name.route.tsxto use new client methods.workflow_evaluations.server.ts.index.tsto include new TypeScript bindings.This description was created by
for dbb1834. You can customize this summary. It will automatically update as commits are pushed.