Repository navigation
docs: add Phoenix Grove (pgsgrove) model pricing entries - #40755
Phoenix-Grove-Systems wants to merge 1 commit into
Conversation
Greptile SummaryAdds Phoenix Grove model metadata to LiteLLM’s primary and backup pricing catalogs.
Confidence Score: 4/5The PR is not safe to merge until The catalogs themselves remain synchronized and valid JSON, but their new entries reference a provider absent from both supported registration mechanisms, making the advertised models unusable through normal prefixed requests. Files Needing Attention: model_prices_and_context_window.json, litellm/model_prices_and_context_window_backup.json
|
| Filename | Overview |
|---|---|
| model_prices_and_context_window.json | Adds the Phoenix Grove catalog, but its provider identifier cannot be resolved by the current provider-routing infrastructure. |
| litellm/model_prices_and_context_window_backup.json | Mirrors the primary catalog additions, including the same unresolved Phoenix Grove provider identifier. |
Reviews (1): Last reviewed commit: "docs: add Phoenix Grove (pgsgrove) model..." | Re-trigger Greptile
| "input_cost_per_token": 6.5e-07, | ||
| "output_cost_per_token": 2e-06, | ||
| "cache_read_input_token_cost": 1.2e-07, | ||
| "litellm_provider": "pgsgrove", |
There was a problem hiding this comment.
The new catalog entries set litellm_provider to pgsgrove, but pgsgrove is absent from both the supported provider list and the JSON provider registry. When a user requests a newly advertised model such as pgsgrove/glm-5.2, provider resolution cannot recognize the prefix and raises BadRequestError instead of routing the request. Register the provider alongside these entries so the new models are usable. The backup catalog contains the same unresolved provider identifier.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Thanks. On the "provider is not registered" point: that is intentional for this PR. |
f7c3791 to
891d46b
Compare
Registers the pgsgrove provider set per main's new test_every_cost_map_provider_is_registered requirement: pgsgrove_models in litellm/__init__.py, populated in _populate_provider_model_sets and listed in _build_models_by_provider.
891d46b to
5184bab
Compare
|
Gentle bump for a code-owner look when convenient. The branch was rebuilt on current main (no conflicts), entries refreshed to the current catalog (36 models), and the provider set is now registered in |
TLDR
Problem this solves:
How it solves it:
pgsgrove/<model>entries to the cost map (35 chat + 1 embedding)pgsgrove_modelsinlitellm/__init__.py, populated in_populate_provider_model_sets, listed in_build_models_by_provider) as required by main'stest_every_cost_map_provider_is_registered.User Flow
Before: a developer pointing LiteLLM at Phoenix Grove sees every call logged at zero spend
litellm.completion(model="openai/glm-5.3-flash", api_base="https://api.pgsgrove.com/v1", api_key="pgsk_...")usagetoken countslitellm.completion_cost(response)returns 0 and https://litellm-domain/ui/?page=logs shows the request at $0After: the same call is priced from the cost map
litellm.completion(model="openai/glm-5.3-flash", api_base="https://api.pgsgrove.com/v1", api_key="pgsk_...")and passcustom_llm_provider="pgsgrove"(or look the model up aspgsgrove/glm-5.3-flash)usagetoken countslitellm.completion_cost(response)returns the per-token cost from the new entry and the logs page shows non-zero spendRelevant issues
None. First contribution for this provider.
Pre-Submission checklist
model_prices_and_context_window.schema.jsonandci_cd/cost_map_guard.pypasses on the PR head (run locally: "cost map guard passed (human PR, file checks only)")python ci_cd/generate_model_prices_schema.py --checkreports the schema in sync and the map validatingScreenshots / Proof of Fix
Before (dfab479)
python -c "import litellm; print(litellm.model_cost.get('pgsgrove/glm-5.3-flash'))"→NoneAfter (730fd00)
python -c "import litellm; print(litellm.model_cost['pgsgrove/glm-5.3-flash']['input_cost_per_token'])"→1e-07python ci_cd/cost_map_guard.py --base HEAD~1 --head HEAD --head-ref docs/add-pgsgrove-pricing→cost map guard passed (human PR, file checks only)cmp model_prices_and_context_window.json litellm/model_prices_and_context_window_backup.json→ identicalNote on the Greptile "provider is not registered" finding:
pgsgroveis a metadata-only provider slug, the same pattern as the existinggmi,llamagate,crusoeandsarvamentries in this file. Today users reach the endpoint withopenai/<model>+api_base, and these entries givecompletion_cost/get_model_infothe pricing. A nativepgsgroveprovider integration would be a separate, later PR.