-
Notifications
You must be signed in to change notification settings - Fork 822
Replace default o3-mini with o4-mini #1710
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
Replace default o3-mini with o4-mini #1710
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨No code suggestions found for the PR. |
ea5c9b2
to
e465682
Compare
model="o3-mini" | ||
model="o4-mini" | ||
fallback_models=["gpt-4o-2024-11-20"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/implement change fallback model to 'gpt-4.1'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Implementation 🛠️
Implementation: Update the fallback model from "gpt-4o-2024-11-20" to "gpt-4.1" as requested.
model="o3-mini" | |
model="o4-mini" | |
fallback_models=["gpt-4o-2024-11-20"] | |
model="o4-mini" | |
fallback_models=["gpt-4.1"] |
See review comment here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe leave it in another change will be cleaner 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if we are updating to more modern models, I do want to remove the 'gpt-4o' from the config.
it looks outdated and unmaintained.
And its just the fallback model (that should not be called in normal usage)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the suggestion. I see the importance of updating to modern models. Would you mind if I submit a separate PR for this change? It would help keep the commits cleaner and make it easier for maintainers to track changes. What do you think?
Thanks for the PR. Please address the comments above, and we can merge |
Sure, I will be back with the updates soon. |
e465682
to
e854383
Compare
User description
We should change the default setting from
o3-mini
too4-mini
to take advantage of the same price while benefiting from improved performance and capability.PR Type
Enhancement, Documentation
Description
Updated default model from
o3-mini
too4-mini
across documentation and configuration.Adjusted references to reflect the new default model in usage guides and examples.
Ensured consistency in model naming in all relevant files.
Changes walkthrough 📝
index.md
Updated model reference in Chrome extension documentation
docs/docs/chrome-extension/index.md
o3-mini
witho4-mini
in extension description.locally.md
Updated model reference in local installation guide
docs/docs/installation/locally.md
o3-mini
too4-mini
.pr_agent_pro.md
Updated model references in PR-Agent Pro overview
docs/docs/overview/pr_agent_pro.md
o3-mini
witho4-mini
in model selection section.changing_a_model.md
Updated default model in model-changing guide
docs/docs/usage-guide/changing_a_model.md
o3-mini
too4-mini
.qodo_merge_models.md
Updated supported models and configuration examples
docs/docs/usage-guide/qodo_merge_models.md
o3-mini
witho4-mini
in supported models list.configuration.toml
Updated default model in configuration file
pr_agent/settings/configuration.toml
o3-mini
too4-mini
.