Skip to content

Conversation

uzaxirr
Copy link
Contributor

@uzaxirr uzaxirr commented Sep 24, 2025

Title

LiteLLM can now support GPT 5 code models.

Relevant issues

fixes: #14838

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • I have added a screenshot of my new test passing locally
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

Changes

  • Added gpt-5-codex and gpt-5-codex-latest to model pricing database
  • i also updated GPT-5 transformation classes to identify Codex variants

Copy link

vercel bot commented Sep 24, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
litellm Error Error Sep 25, 2025 1:34am

@uzaxirr
Copy link
Contributor Author

uzaxirr commented Sep 24, 2025

Screenshot 2025-09-24 at 10 23 11 AM

@ishaan-jaff @krrishdholakia
no idea why vercel deployment is failing, im unable to see the logs as well

"output_cost_per_token_flex": 5e-06,
"output_cost_per_token_priority": 2e-05,
"supported_endpoints": [
"/v1/chat/completions",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Completions is not supported, see https://platform.openai.com/docs/models/gpt-5-codex

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@uzaxirr please fix this ^

Copy link
Contributor

@ishaan-jaff ishaan-jaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reviewed @uzaxirr

"output_cost_per_token_flex": 5e-06,
"output_cost_per_token_priority": 2e-05,
"supported_endpoints": [
"/v1/chat/completions",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@uzaxirr please fix this ^

@uzaxirr
Copy link
Contributor Author

uzaxirr commented Sep 25, 2025

Screenshot 2025-09-25 at 7 04 39 AM
Screenshot 2025-09-25 at 7 05 10 AM

@ishaan-jaff I have fixed the above-mentioned issue and also tested the change locally. Let me know if you still need any more changes.

Copy link
Contributor

@ishaan-jaff ishaan-jaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ishaan-jaff ishaan-jaff merged commit fcfe856 into BerriAI:main Oct 1, 2025
5 of 7 checks passed
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.

[Feature]: Support a new model - gpt-5-codex
3 participants