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: Upgraded Azure OpenAI SDK version. #10487

Closed
wants to merge 2 commits into from

Conversation

shethaadit
Copy link
Contributor

@shethaadit shethaadit commented Feb 11, 2025

Description

Upgraded Azure OpenAI SDK version.

Fixes #10484

@shethaadit shethaadit requested a review from a team as a code owner February 11, 2025 17:59
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label Feb 11, 2025
@github-actions github-actions bot changed the title Upgraded Azure OpenAI SDK version. .Net: Upgraded Azure OpenAI SDK version. Feb 11, 2025
@shethaadit
Copy link
Contributor Author

Hi @markwallace-microsoft / @RogerBarreto, could you please approve the build run? I would like to see if the build gets passed since my local set up is broken.

@dmytrostruk
Copy link
Member

@shethaadit I approved build run, it looks like there are some errors, here are a couple of them:

Error: /home/runner/work/semantic-kernel/semantic-kernel/dotnet/src/Agents/OpenAI/Internal/AssistantThreadActions.cs(764,69): error CS0103: The name 'RunStepType' does not exist in the current context [/home/runner/work/semantic-kernel/semantic-kernel/dotnet/src/Agents/OpenAI/Agents.OpenAI.csproj::TargetFramework=netstandard2.0]
Error: /home/runner/work/semantic-kernel/semantic-kernel/dotnet/src/Agents/OpenAI/Internal/AssistantThreadActions.cs(770,98): error CS1061: 'RunStepToolCall' does not contain a definition for 'ToolCallId' and no accessible extension method 'ToolCallId' accepting a first argument of type 'RunStepToolCall' could be found (are you missing a using directive or an assembly reference?) [/home/runner/work/semantic-kernel/semantic-kernel/dotnet/src/Agents/OpenAI/Agents.OpenAI.csproj::TargetFramework=netstandard2.0]

@dmytrostruk
Copy link
Member

@shethaadit Please see this commit 6f0b6f0, it contains required changes to align with introduced breaking changes from latest OpenAI SDK release.

@shethaadit
Copy link
Contributor Author

@shethaadit Please see this commit 6f0b6f0, it contains required changes to align with introduced breaking changes from latest OpenAI SDK release.

Thank you @dmytrostruk for the commit.

@RogerBarreto
Copy link
Member

@shethaadit Thanks for the proposed PR.

As @dmytrostruk also noted I have create another one including some already missing features we need to consider during the upgrade:

I'm closing this PR as the above covers it. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.NET Issue or Pull requests regarding .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

.Net: Update Both (OpenAI + AzureOpenAI) Connectors to use 2.2.0-beta.1 version.
4 participants