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

Skip to content

Conversation

@shuyangli
Copy link
Member

@shuyangli shuyangli commented Dec 21, 2025

Renames some routes that only return count information from -stats to -count, so it's more clear when the routs actually return statistics about e.g. metrics.

Cleanup only.


Important

Renamed internal routes and related code from -stats to -count for clarity and consistency across the codebase.

  • Routes:
    • Renamed internal routes from -stats to -count in internal.rs.
    • Updated route paths in get_inference_count_handler and get_inference_with_feedback_count_handler.
  • Functions:
    • Renamed functions in inference_count.rs to reflect -count instead of -stats.
    • Updated function calls in get_inference_count() and get_inference_with_feedback_count().
  • Types:
    • Renamed types from InferenceStats* to InferenceCount* in inference_count.rs.
    • Updated type imports and usage in various files including tensorzero.ts and inference.server.ts.
  • Tests:
    • Updated test cases to reflect new route and function names in inference_count_queries.rs and inference_count.rs.

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

@shuyangli shuyangli requested a review from virajmehta December 21, 2025 13:20
@shuyangli shuyangli mentioned this pull request Dec 21, 2025
52 tasks
@shuyangli shuyangli assigned shuyangli and unassigned virajmehta Dec 21, 2025
@shuyangli shuyangli removed the request for review from virajmehta December 21, 2025 13:23
@shuyangli
Copy link
Member Author

This one conflicts so will do later

Copy link
Member Author

fixed merge conflicts

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/20441633256

@graphite-app
Copy link

graphite-app bot commented Dec 23, 2025

Merge activity

  • Dec 23, 4:30 AM UTC: This pull request can not be added to the Graphite merge queue. Please try rebasing and resubmitting to merge when ready.
  • Dec 23, 4:31 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..

@shuyangli shuyangli force-pushed the sl/rename-stats-to-count-routes branch from faa63f2 to c3ae88f Compare December 23, 2025 10:30
@shuyangli shuyangli force-pushed the sl/rename-stats-to-count-routes branch 3 times, most recently from 50cd499 to 1e68be1 Compare December 23, 2025 11:56
@shuyangli shuyangli force-pushed the sl/rename-stats-to-count-routes branch from 1e68be1 to dbc7866 Compare December 23, 2025 12:37
@GabrielBianconi GabrielBianconi added this pull request to the merge queue Dec 23, 2025
Merged via the queue into main with commit 9379ca8 Dec 23, 2025
56 checks passed
@GabrielBianconi GabrielBianconi deleted the sl/rename-stats-to-count-routes branch December 23, 2025 16:21
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