-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add GPT-5 OpenAI model strings #2477
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…gpt-5-mini, gpt-5-nano and 2025-08-07 snapshots
…el list for KnownModelName parity
…ixed and bare GPT-5 IDs in KnownModelName
6617a52
to
8b170ea
Compare
The new openai sdk (1.99.3) breaks some stuff, so I didn't bump it: https://pypi.org/project/openai/ |
Would it make sense to introduce the new |
yes |
# Include latest GPT-5 variants until the SDK enumerates them | ||
Literal[ | ||
'gpt-5', | ||
'gpt-5-mini', | ||
'gpt-5-nano', | ||
'gpt-5-2025-08-07', | ||
'gpt-5-mini-2025-08-07', | ||
'gpt-5-nano-2025-08-07', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we just need to bump the minimum.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Kludex Bumping the min breaks a bunch of tests
Got it. |
Hi, Could you also add the chat model? |
Responding to PR feedbacK
|
Adds GPT-5 model IDs to KnownModelName