Skip to content

Azure o3/o4 tool support #5153

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 3 commits into from
Apr 23, 2025
Merged

Conversation

muliyul
Copy link
Contributor

@muliyul muliyul commented Apr 14, 2025

Description

Add tool support for GPT o3/o4 azure deployed models.

Checklist

  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screenshots

[ For visual changes, include screenshots. Screen recordings are particularly helpful, and appreciated! ]

Testing instructions

[ For new or modified features, provide step-by-step testing instructions to validate the intended behavior of the change, including any relevant tests to run. ]

@muliyul muliyul requested a review from a team as a code owner April 14, 2025 16:06
@muliyul muliyul requested review from sestinj and removed request for a team April 14, 2025 16:06
Copy link

netlify bot commented Apr 14, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 7557bf2
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67fd6070f3d2b20008e85442

Copy link
Contributor

@sestinj sestinj left a comment

Choose a reason for hiding this comment

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

@muliyul this looks good, though if you want to add o4 support I think this PR only does o3—what that intentional? I'm going to merge, but feel free to open another PR

@sestinj sestinj merged commit 9e6890e into continuedev:main Apr 23, 2025
31 checks passed
@muliyul
Copy link
Contributor Author

muliyul commented Apr 23, 2025

I thought gpt-4 prefix is sufficient for o4 & o4 mini. isn't that true?

Also while we're at it, can we release this to jetbrains ides?

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