-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
.Net: Update {Azure}OpenAI Connectors to latest 2.2.0-beta.1 #10496
Conversation
@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! |
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.
LGTM but PR is failing required checks.
dotnet/src/SemanticKernel.Abstractions/AI/ChatCompletion/ChatHistory.cs
Outdated
Show resolved
Hide resolved
dotnet/src/SemanticKernel.Abstractions/AI/ChatCompletion/ChatHistory.cs
Outdated
Show resolved
Hide resolved
dotnet/src/Connectors/Connectors.OpenAI/Settings/OpenAIPromptExecutionSettings.cs
Show resolved
Hide resolved
dotnet/src/SemanticKernel.Abstractions/AI/ChatCompletion/ChatHistory.cs
Outdated
Show resolved
Hide resolved
dotnet/src/SemanticKernel.Abstractions/CompatibilitySuppressions.xml
Outdated
Show resolved
Hide resolved
Very nice to see that the effort is supported! Awesome and looking forward to have this released ASAP :) |
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.
system
anddeveloper
roles, only acceptinguser
.reasoning_effort
parameter. (o1-mini)