Skip to content

Use AzureOpenAI in OpenAIWrapper#868

Closed
yiranwu0 wants to merge 2 commits into
mainfrom
useazureai
Closed

Use AzureOpenAI in OpenAIWrapper#868
yiranwu0 wants to merge 2 commits into
mainfrom
useazureai

Conversation

@yiranwu0

@yiranwu0 yiranwu0 commented Dec 4, 2023

Copy link
Copy Markdown
Contributor

Why are these changes needed?

Creating AzureOpenAI in OpenAIWrapper if api_type is "azure"

Related issue number

Checks

@sonichi

sonichi commented Jan 5, 2024

Copy link
Copy Markdown
Contributor

@kevin666aa if you'd like help with this PR, I'm happy to find helpers from discord.

@sonichi sonichi mentioned this pull request Jan 5, 2024
14 tasks
@yiranwu0

yiranwu0 commented Jan 5, 2024

Copy link
Copy Markdown
Contributor Author

@kevin666aa if you'd like help with this PR, I'm happy to find helpers from discord.

Yes, thanks!

@cforce

cforce commented Mar 3, 2024

Copy link
Copy Markdown

@kevin666aa can you please rebase and and check tests afterwards 🙏

@yiranwu0

yiranwu0 commented Mar 3, 2024

Copy link
Copy Markdown
Contributor Author

@cforce This PR should be closed. The described feature is implemented in #1232.

@yiranwu0 yiranwu0 closed this Mar 3, 2024
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.

3 participants