-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
feat(OpenAI Chat Model Node, OpenAI Node): Include o3 models in model selection #13005
feat(OpenAI Chat Model Node, OpenAI Node): Include o3 models in model selection #13005
Conversation
…models-to-include-o3
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
✅ All Cypress E2E specs passed |
n8n
|
Project |
n8n
|
Branch Review |
ai-658-adjust-the-filtering-of-the-openai-models-to-include-o3
|
Run status |
|
Run duration | 04m 24s |
Commit |
|
Committer | JP van Oosten |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
1
|
|
5
|
|
0
|
|
433
|
View all changes introduced in this branch ↗︎ |
Got released with |
Thank you! please add the ability to modify reasoning effort! there is massive performance variance between the different options! :) |
please add modify reasoning effort +1 |
yes, please add reasoning effort. openAI defaults to "medium" if no reasoning effort is specified. From openAI documentation: reasoning_effort Optional Constrains effort on reasoning for reasoning models. Currently supported values are low, medium, and high. Reducing reasoning effort can result in faster responses and fewer tokens used on reasoning in a response. Ref: |
Summary
The model-filters didn't include the o3 models yet. This change fixes that. This allows you to use o3-mini now.
Related Linear tickets, Github issues, and Community forum posts
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)