Skip to content

demo(human): drop a model from the root map only - #39830

Closed
mateo-berri wants to merge 1 commit into
litellm_cost_map_guardfrom
litellm_cost_map_demo_human
Closed

mateo-berri wants to merge 1 commit into
litellm_cost_map_guardfrom
litellm_cost_map_demo_human

Conversation

@mateo-berri

Copy link
Copy Markdown
Contributor

Throwaway demo PR for the cost map guard (LIT-6995). Closed once the check result is captured.

@greptile-apps

greptile-apps Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This demo removes one OpenRouter model from the root cost map

  • The bundled backup remains unchanged
  • Existing metadata and pricing lookups for the removed OpenRouter key lose their exact mapping

Confidence Score: 4/5

This PR should not merge because existing lookups for the removed OpenRouter model can no longer resolve its metadata and pricing

The active cost map loses the only matching OpenRouter entry, while lookup code does not consult the bundled backup per model

Files Needing Attention: model_prices_and_context_window.json

Important Files Changed

Filename Overview
model_prices_and_context_window.json Removes an active OpenRouter model mapping, leaving existing callers without matching metadata or pricing

Comments Outside Diff (1)

  1. model_prices_and_context_window.json, line 62986-62999 (link)

    P1 OpenRouter mapping removed

    Requests using openrouter/openai/gpt-3.5-turbo-instruct can no longer resolve model metadata or pricing, causing lookup failures for this existing model

Reviews (1): Last reviewed commit: "demo(human): drop a model from the root ..." | Re-trigger Greptile

@codecov

codecov Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mateo-berri

Copy link
Copy Markdown
Contributor Author

Throwaway guard demo: pull_request_target only runs from the default branch, so the live demo happens once the guard lands there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant