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

.Net: Update {Azure}OpenAI Connectors to latest 2.2.0-beta.1 #10496

Merged

Conversation

RogerBarreto
Copy link
Member

Motivation and Context

Some Azure Backend API's may not fully work with the latest SDK updates, but this is rather a limitation of the backend not related to the SDK itself.

  • Some model and Azure OpenAI endpoints trigger error for both system and developer roles, only accepting user.
  • Attempting to use temperature parameters in reasoning models (o1+) is not supported in the backends
  • Some backend models may still reject the reasoning_effort parameter. (o1-mini)

@RogerBarreto RogerBarreto self-assigned this Feb 12, 2025
@RogerBarreto RogerBarreto requested a review from a team as a code owner February 12, 2025 00:02
@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel kernel.core labels Feb 12, 2025
@dmytrostruk
Copy link
Member

@RogerBarreto FYI there is already existing community PR to update {Azure}OpenAI package versions: #10487

But it looks like your PR has more changes, feel free to decide which PR should be merged. Thanks!

@rwjdk
Copy link

rwjdk commented Feb 12, 2025

It makes sense that 01-mini will not work... The model version is still pre-full release (It's from September last year)

image

Copy link
Member

@markwallace-microsoft markwallace-microsoft left a comment

Choose a reason for hiding this comment

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

LGTM but PR is failing required checks.

@RogerBarreto RogerBarreto added this pull request to the merge queue Feb 12, 2025
@joslat
Copy link
Contributor

joslat commented Feb 12, 2025

Very nice to see that the effort is supported! Awesome and looking forward to have this released ASAP :)

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 12, 2025
@RogerBarreto RogerBarreto added this pull request to the merge queue Feb 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 12, 2025
@dmytrostruk dmytrostruk added this pull request to the merge queue Feb 12, 2025
Merged via the queue into microsoft:main with commit 5e208a7 Feb 12, 2025
19 checks passed
@RogerBarreto RogerBarreto deleted the issues/10484-openai-versionbump branch February 13, 2025 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel.core kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code
Projects
None yet
6 participants