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

Skip to content

Conversation

@shuyangli
Copy link
Member

@shuyangli shuyangli commented Dec 22, 2025

Important

Add datapoint_id filter to get_evaluation_results for more specific queries, with extensive test coverage.

  • Behavior:
    • Add datapoint_id filter to get_evaluation_results in evaluation_queries.rs, allowing queries for specific datapoints.
    • Update get_evaluation_results_handler in get_evaluation_results.rs to handle datapoint_id.
    • Modify TensorZeroClient.getEvaluationResults in tensorzero.ts to support datapointId option.
  • Tests:
    • Add tests for get_evaluation_results with datapoint_id in evaluation_queries.rs and get_evaluation_results.rs.
    • Add E2E tests in evaluation_queries.rs to verify datapoint_id filtering.
  • Misc:
    • Update TypeScript client in evaluations.server.ts to use new datapointId filter.

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

@shuyangli shuyangli marked this pull request as ready for review December 22, 2025 20:38
@shuyangli shuyangli force-pushed the sl/migrate-get-evaluations-for-datapoint branch 2 times, most recently from fbaa5af to 4e6135f Compare December 22, 2025 20:49
Copy link
Member Author

/merge-queue

@github-actions
Copy link
Contributor

πŸš€ Merge queue workflow triggered!

View the run: https://github.com/tensorzero/tensorzero/actions/runs/20443626313

@shuyangli shuyangli mentioned this pull request Dec 22, 2025
52 tasks
Investigate why test is failing
@shuyangli shuyangli force-pushed the sl/migrate-get-evaluations-for-datapoint branch from 4e6135f to a75a1ca Compare December 22, 2025 21:43
@shuyangli
Copy link
Member Author

/merge-queue

@github-actions
Copy link
Contributor

πŸš€ Merge queue workflow triggered!

View the run: https://github.com/tensorzero/tensorzero/actions/runs/20444781475

@virajmehta virajmehta added this pull request to the merge queue Dec 23, 2025
Merged via the queue into main with commit 0794a7f Dec 23, 2025
61 checks passed
@virajmehta virajmehta deleted the sl/migrate-get-evaluations-for-datapoint branch December 23, 2025 03:49
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