Repository navigation
chore: Deprecate ai.monitoring.record_token_usage() - #7918
Conversation
01211f3 to
75d5882
Compare
75d5882 to
6f26527
Compare
Co-authored-by: Ivana Kellyer <[email protected]>
Codecov Results 📊✅ 133451 passed | ⏭️ 7276 skipped | Total: 140727 | Pass Rate: 94.83% | Execution Time: 460m 47s 📊 Comparison with Base Branch
All tests are passing successfully. ✅ Patch coverage is 85.71%. Project has 2593 uncovered lines. Files with missing lines (6)
Coverage diff@@ Coverage Diff @@
## master #PR +/-##
==========================================
- Coverage 90.37% 90.26% -0.11%
==========================================
Files 202 202 —
Lines 26552 26625 +73
Branches 9854 9926 +72
==========================================
+ Hits 23995 24032 +37
- Misses 2557 2593 +36
- Partials 1495 1515 +20Generated by Codecov Action |
ericapisani
left a comment
There was a problem hiding this comment.
These changes LGTM but warden looks to be right about huggingface_hub still using record_token_usage. Is the migration away from this there happening in a follow up?
3a00c3d to
a8818f9
Compare
a8818f9 to
8238921
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 8238921. Configure here.
8238921 to
cb7a639
Compare

Description
Deprecate the function, pending removal in the upcoming major.
Inline uses of the function in integrations.
Issues
Contributes to #6894
Reminders
uv run ruff.feat:,fix:,ref:,meta:)