Skip to content
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: add support for o1 high reasoning #9605

Merged
merged 2 commits into from
Dec 23, 2024
Merged

Conversation

fontanierh
Copy link
Contributor

Description

  • adds a new openai_o1_high_reasoning_feature feature flag
  • adds a new o1-high-resoning global agent

The new global agent is the same as the o1 global agent, but with reasoning_effort set to high. This assistant is only available to workspaces that have the openai_o1_high_reasoning_feature FF enabled.

Risk

Still in the critical path, but risk relatively low (+ well tested)

Deploy Plan

deploy front

supportsVision: false,
supportsVision: true,
};
export const O1_HIGH_REASONING_MODEL_CONFIG: ModelConfigurationType = {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm making it a separate "model config", because will be adding the O1 models to assistant builder. I don't want to expose configuring the reasoning_effort from the UI for now, so it's simpler to expose as a separate model

@fontanierh fontanierh force-pushed the feat/o1-high-reasoning branch from cbb66d0 to 979cfb5 Compare December 23, 2024 11:14
Copy link
Contributor

@philipperolet philipperolet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@fontanierh fontanierh merged commit 47f3ed7 into main Dec 23, 2024
7 checks passed
@fontanierh fontanierh deleted the feat/o1-high-reasoning branch December 23, 2024 12:56
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.

2 participants