-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[ENH]: add request timing to metering #4877
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
base: main
Are you sure you want to change the base?
Conversation
…per/chroma-metering-macros-integration
…ithub.com/chroma-core/chroma into cooper/request-latencies
Reviewer ChecklistPlease leverage this checklist to ensure your code review is thorough before approving Testing, Bugs, Errors, Logs, Documentation
System Compatibility
Quality
|
Integrate New Metering Library and Add Request Execution Timing to Frontend This PR replaces all prior metering event collection and emission logic in the Rust frontend with the new Key Changes: Affected Areas: This summary was automatically generated by @propel-code-bot |
…per/chroma-metering-macros-integration
…ithub.com/chroma-core/chroma into cooper/request-latencies
…per/chroma-metering-macros-integration
…ithub.com/chroma-core/chroma into cooper/request-latencies
…per/chroma-metering-macros-integration
…ithub.com/chroma-core/chroma into cooper/request-latencies
…per/chroma-metering-macros-integration
…ithub.com/chroma-core/chroma into cooper/request-latencies
…per/chroma-metering-macros-integration
…ithub.com/chroma-core/chroma into cooper/request-latencies
This PR uses the
chroma-metering
library introduced in #4868 to add execution timing to requests received on the frontend.pytest
for python,yarn test
for js,cargo test
for rustDocumentation Changes
N/A