Skip to content

Bugfix | Fixed a bug when calling reasoning models with store=False #920

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

Merged
merged 1 commit into from
Jun 24, 2025

Conversation

niv-hertz
Copy link
Contributor

@niv-hertz niv-hertz commented Jun 23, 2025

resolves #919

@rm-openai
Copy link
Collaborator

Thanks - though can you actually add includes to the ModelSettings? Because its possible you use e.g. o3 but don't set model_settings.reasoning, which would still cause an error

@niv-hertz
Copy link
Contributor Author

Thanks - though can you actually add includes to the ModelSettings? Because its possible you use e.g. o3 but don't set model_settings.reasoning, which would still cause an error

You're right, changed per your suggestion :)

@rm-openai
Copy link
Collaborator

@niv-hertz did you mean to close this?

@niv-hertz
Copy link
Contributor Author

@rm-openai - Nope, not sure why is it closed

@niv-hertz

This comment was marked as outdated.

@niv-hertz

This comment was marked as outdated.

@niv-hertz

This comment was marked as outdated.

@niv-hertz niv-hertz reopened this Jun 24, 2025
@niv-hertz
Copy link
Contributor Author

@rm-openai - Re-opened with your requested fixes :)

@rm-openai rm-openai merged commit d88bf14 into openai:main Jun 24, 2025
10 checks passed
niv-hertz added a commit to coralogix/openai-agents-python that referenced this pull request Jun 24, 2025
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.

Bug | Reasoning on OpenAI's models doesn't work with store=False
2 participants