Skip to content

Fix misleading 'maximize' high effort description on xhigh models #7874

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.

Sign up for GitHub

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

Jump to bottom
Merged
etraut-openai merged 1 commit into from
Dec 12, 2025
Merged

Fix misleading 'maximize' high effort description on xhigh models #7874

etraut-openai merged 1 commit into from
Dec 12, 2025
+13 −13

Conversation

@voctory
Copy link
Contributor

@voctory voctory commented Dec 11, 2025 •
edited
Loading

Notes

  • switch misleading High reasoning effort descriptions from "Maximizes reasoning depth" to "Higher reasoning depth" across models with xhigh reasoning. Affects GPT-5.1 Codex Max and Robin
  • refresh model list fixtures and chatwidget snapshots to match new copy

Revision

  • R2: Change 'Higher' to 'Greater'
  • R1: Initial

Testing

image image

@voctory
Copy link
Contributor Author

voctory commented Dec 11, 2025

@codex r

@chatgpt-codex-connector
Copy link
Contributor

chatgpt-codex-connector bot commented Dec 11, 2025

Codex Review: Didn't find any major issues. Nice work!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@etraut-openai
Copy link
Collaborator

etraut-openai commented Dec 11, 2025

Thanks for the contribution. I don't think "higher" is quite the right word in this context. The adjective applies to the word "depth", and "higher depth" doesn't really sound right. Maybe "Greater" would be a better choice?

@etraut-openai etraut-openai added the needs-response Additional information is requested label Dec 11, 2025
@voctory
Fix misleading 'maximize' high effort description on xhigh models
0b59870 
- switch High reasoning effort descriptions to "Higher reasoning depth" across models with xhigh reasoning effort instead of "Maximizes reasoning depth". Affects GPT-5.1 Codex Max and Robin
- refresh model list fixtures and chatwidget snapshots to match new copy
@voctory
Copy link
Contributor Author

voctory commented Dec 12, 2025

@codex r

@chatgpt-codex-connector
Copy link
Contributor

chatgpt-codex-connector bot commented Dec 12, 2025

Codex Review: Didn't find any major issues. Chef's kiss.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@voctory
Copy link
Contributor Author

voctory commented Dec 12, 2025

@etraut-openai Thanks for catching that. I've updated it to "Greater"

@etraut-openai etraut-openai removed the needs-response Additional information is requested label Dec 12, 2025
@etraut-openai etraut-openai merged commit 95f7d37 into openai:main Dec 12, 2025
26 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 12, 2025
@voctory voctory deleted the fix-maximize-desc branch December 12, 2025 00:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@etraut-openai etraut-openai etraut-openai approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@voctory @etraut-openai