Skip to content

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