Fix: add OpenRouter Qwen 3.6 Plus metadata - #27486
Conversation
Co-authored-by: ishaan-berri <ishaan-berri@users.noreply.github.com>
|
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
f2e9738
into
BerriAI:litellm_internal_staging
Greptile SummaryThis PR adds metadata for the
Confidence Score: 4/5Safe to merge; the change is additive metadata only and does not touch any request-path code. The new JSON entry is data-only and all values check out against the OpenRouter listing. The sole issue is that the entry is inserted before No files require special attention; the two JSON files are straightforward metadata additions.
|
| Filename | Overview |
|---|---|
| model_prices_and_context_window.json | Adds openrouter/qwen/qwen3.6-plus entry; pricing ($0.325/M in, $1.95/M out), context window (1M), and max output (65536) all verified against the OpenRouter listing. Entry is inserted slightly out of alphabetical order relative to adjacent qwen3.5 keys. |
| litellm/model_prices_and_context_window_backup.json | Mirrors the same openrouter/qwen/qwen3.6-plus entry added to the primary JSON; identical values, same ordering concern. |
| tests/test_litellm/test_cost_calculator.py | New test verifies all fields of the new entry by loading the local cost map (no network calls); consistent with the established pattern in the file. |
Reviews (1): Last reviewed commit: "Add OpenRouter Qwen 3.6 Plus metadata" | Re-trigger Greptile
| "openrouter/qwen/qwen3.6-plus": { | ||
| "input_cost_per_token": 3.25e-07, | ||
| "litellm_provider": "openrouter", | ||
| "max_input_tokens": 1000000, | ||
| "max_output_tokens": 65536, | ||
| "max_tokens": 65536, | ||
| "mode": "chat", | ||
| "output_cost_per_token": 1.95e-06, | ||
| "source": "https://openrouter.ai/qwen/qwen3.6-plus", | ||
| "supports_function_calling": true, | ||
| "supports_reasoning": true, | ||
| "supports_tool_choice": true, | ||
| "supports_vision": true | ||
| }, | ||
| "openrouter/qwen/qwen3.5-35b-a3b": { |
There was a problem hiding this comment.
The new
qwen3.6-plus entry is inserted immediately before openrouter/qwen/qwen3.5-35b-a3b, placing a higher-numbered model before a lower-numbered one. If this file is maintained in alphabetical key order (as most entries appear to be), the new key should come after qwen3.5-* entries since "3.6" > "3.5" lexicographically. This doesn't affect runtime behavior, but keeping the file sorted makes future diffs and lookups easier.
| "openrouter/qwen/qwen3.6-plus": { | |
| "input_cost_per_token": 3.25e-07, | |
| "litellm_provider": "openrouter", | |
| "max_input_tokens": 1000000, | |
| "max_output_tokens": 65536, | |
| "max_tokens": 65536, | |
| "mode": "chat", | |
| "output_cost_per_token": 1.95e-06, | |
| "source": "https://openrouter.ai/qwen/qwen3.6-plus", | |
| "supports_function_calling": true, | |
| "supports_reasoning": true, | |
| "supports_tool_choice": true, | |
| "supports_vision": true | |
| }, | |
| "openrouter/qwen/qwen3.5-35b-a3b": { | |
| "openrouter/qwen/qwen3.5-35b-a3b": { |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Co-authored-by: oss-agent-shin <279349115+oss-agent-shin@users.noreply.github.com> Co-authored-by: ishaan-berri <ishaan-berri@users.noreply.github.com>
Summary
Adds the missing OpenRouter
qwen/qwen3.6-plusmodel metadata to both LiteLLM cost-map files, including context length, output limit, pricing, source URL, and supported capability flags. This fixes local cost-map lookups returning no model info for the new OpenRouter Qwen 3.6 Plus model.Repro
A local cost-map lookup for
openrouter/qwen/qwen3.6-pluspreviously returnedNone, so cost/context information was unavailable for that OpenRouter model.Evidence
Terminal evidence: qwen36_model_metadata_test_evidence.txt
Tests
pytest tests/test_litellm/test_cost_calculator.py::test_openrouter_qwen36_plus_model_info -qfailed withassert None is not None.tests/test_litellm/test_cost_calculator.py::test_openrouter_qwen36_plus_model_info.pytest tests/test_litellm/test_cost_calculator.py::test_openrouter_qwen36_plus_model_info -q.pytest tests/test_litellm/test_cost_calculator.py -q(39 passed, 12 warnings).CI
Review
Relevant issues
Fixes #27424
Linear ticket
N/A
Pre-Submission checklist
tests/test_litellm/directory, Adding at least 1 test is a hard requirement - see detailsmake test-unit@greptileaiand received a Confidence Score of at least 4/5 before requesting a maintainer reviewDelays in PR merge?
N/A
CI (LiteLLM team)
Branch creation CI run
Link:
CI run for the last commit
Link: Available PR checks are green on the latest commit.
Merge / cherry-pick CI run
Links:
Screenshots / Proof of Fix
See the terminal evidence and tests above.
Type
🐛 Bug Fix
✅ Test
Changes
openrouter/qwen/qwen3.6-plustomodel_prices_and_context_window.json.litellm/model_prices_and_context_window_backup.json.