Skip to content

fix: Add gpt-5 models to reasoning model detection for temperature parameter handling #2483

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.

Sign up for GitHub

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

Merged
merged 3 commits into from
Aug 11, 2025
Merged

fix: Add gpt-5 models to reasoning model detection for temperature parameter handling #2483

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1d91ca1
fix: Add gpt-5 models to reasoning model detection for temperature pa…
claude[bot] Aug 8, 2025
6ec049c
add test
Kludex Aug 11, 2025
4cdb415
Merge branch 'main' into claude/issue-2482-20250808-1310
Kludex Aug 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pydantic_ai_slim/pydantic_ai/profiles/openai.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class OpenAIModelProfile(ModelProfile):

def openai_model_profile(model_name: str) -> ModelProfile:
"""Get the model profile for an OpenAI model."""
is_reasoning_model = model_name.startswith('o')
is_reasoning_model = model_name.startswith('o') or model_name.startswith('gpt-5')
# Structured Outputs (output mode 'native') is only supported with the gpt-4o-mini, gpt-4o-mini-2024-07-18, and gpt-4o-2024-08-06 model snapshots and later.
# We leave it in here for all models because the `default_structured_output_mode` is `'tool'`, so `native` is only used
# when the user specifically uses the `NativeOutput` marker, so an error from the API is acceptable.
Expand Down