test(litellm): Set LITELLM_LOCAL_MODEL_COST_MAP=true - #7630
Conversation
0504daf to
268d1f9
Compare
Codecov Results 📊✅ 129753 passed | ⏭️ 7148 skipped | Total: 136901 | Pass Rate: 94.78% | Execution Time: 434m 58s 📊 Comparison with Base Branch
➖ Removed Tests (1)View removed tests
All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 2543 uncovered lines. Coverage diff@@ Coverage Diff @@
## master #PR +/-##
==========================================
+ Coverage 90.22% 90.22% —%
==========================================
Files 194 194 —
Lines 25997 25997 —
Branches 9662 9662 —
==========================================
+ Hits 23454 23454 —
- Misses 2543 2543 —
- Partials 1451 1451 —Generated by Codecov Action |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 268d1f9. Configure here.
| # which targets the workspace `.venv` instead of the tox env and hangs. | ||
| ray: RAY_ENABLE_UV_RUN_RUNTIME_ENV=0 | ||
|
|
||
| litellm: LITELLM_LOCAL_MODEL_COST_MAP=true |
There was a problem hiding this comment.
Local cost map flag value ignored
Medium Severity
LiteLLM enables the bundled cost map only when LITELLM_LOCAL_MODEL_COST_MAP equals the string True. The lowercase true set here does not match that check, so litellm test envs still fetch the remote cost map that this change is meant to disable.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 268d1f9. Configure here.
2 participants
Description
Avoid
litellmmaking an HTTP call to determine model costs.This forces
litellmto do a local lookup with a cost map bundled in thelitellmwheel. Lookup failures will then surface in the weekly update.Issues
Reminders
uv run ruff.feat:,fix:,ref:,meta:)