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

Azure OpenAI updates #46103

Merged
merged 14 commits into from
Sep 23, 2024
Merged

Azure OpenAI updates #46103

merged 14 commits into from
Sep 23, 2024

Conversation

ralph-msft
Copy link
Member

  • Nuget package updates:
    • System.ClientModel to version 1.1.0
    • OpenAI to version 2.0.0-beta.12
  • Latest API updates for Azure OpenAI

@azure-sdk
Copy link
Collaborator

azure-sdk commented Sep 20, 2024

API change check

API changes are not detected in this pull request.

Copy link
Member

@annelo-msft annelo-msft left a comment

Choose a reason for hiding this comment

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

How many of the workaround noted in #45996 will we be able to remove as part of this PR? I am very excited about removing those so issues around repo-wide updates like @m-redding's STJ upgrade will be caught earlier in the future.

Specifically, can we remove the workarounds we introduced in:

@benbp as FYI as well

Custom/Internal/AzureAsyncCollectionResult.cs Outdated Show resolved Hide resolved
Copy link
Member

@annelo-msft annelo-msft left a comment

Choose a reason for hiding this comment

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

Parts related to the SCM 1.1.0 upgrade LGTM!

I defer to others on the new public APIs in the library.

@jsquire jsquire dismissed their stale review September 20, 2024 22:26

root files have moved

@ralph-msft ralph-msft requested a review from jsquire September 20, 2024 23:18
@annelo-msft annelo-msft merged commit f5b4dfd into main Sep 23, 2024
60 checks passed
@annelo-msft annelo-msft deleted the ralphe/scm_1_1_0_update branch September 23, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Azure OpenAI: Upgrade TestFramework to STJ 6.0.9
5 participants