Skip to content

Update model information to support the new OpenAI models #368

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
Apr 18, 2025

Conversation

daxian-dbw
Copy link
Member

@daxian-dbw daxian-dbw commented Apr 18, 2025

PR Summary

Fix #360

Update model information to support the new OpenAI models.

  • The OpenAI reasoning models do not support the temperature setting -- they error out if you set the value. But the DeepSeek R1 OpenAI compatible API still allows passing the temperature setting even though it has not effect.

  • I don't have access to the o3 model (requires verifying organization), but I've verified the changes with o1, o3-mini, o4-mini, gpt-4.1, gpt-4.1-mini, and gpt-4.1-nano modules and it works fine.

  • The o1-mini model doesn't work because it doesn't support the system prompt (very weird). Given that the o3-mini is out and recommended, I think it's fine to not work with o1-mini.

Regarding the reasoning/thinking tokens:

We will work on supporting the CoT tokens once .NET OpenAI SDK is fixed to expose the reasoning_content field.

@daxian-dbw daxian-dbw merged commit 7aca7db into PowerShell:main Apr 18, 2025
4 checks passed
@daxian-dbw daxian-dbw deleted the model branch April 18, 2025 18:09
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.

Support OpenAI Reasoning Models
2 participants