Additional cost mappings for gpt-5.1-chat #16769
Conversation
|
Someone is attempting to deploy a commit to the CLERKIEAI Team on Vercel. A member of the Team first needs to authorize it. |
|
IQHL (Hans Jacob Landelius) seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
| "supports_tool_choice": false, | ||
| "supports_vision": true | ||
| }, | ||
| "gpt-5.1-chat": { |
There was a problem hiding this comment.
this isn't the openai model name though - so if i made this request to openai it would fail.
why not just save this as azure/gpt-5.1-chat ? @jlan-nl
There was a problem hiding this comment.
azure/gpt-5.1-chat wasn't there when I created the PR, but you're right that it works just as well :)
I'll close the PR. Thanks!
There was a problem hiding this comment.
Opened an issue that is related to this, please check it out if you get the opportunity @Sameerlite @krrishdholakia
4 participants
Relevant issues
Azure uses the base_model
gpt-5.1-chatinstead ofgpt-5.1-chat-latest, which means we don't get any cost tracking on that model with the current version of the cost mapping.Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
- [ ] I have Added testing in thetests/litellm/directory, Adding at least 1 test is a hard requirement - see details[ ] My PR passes all unit tests onmake test-unitType
🆕 New Feature
Changes
Adde our key to the cost mapping