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

Adds DALL-E note to Azure OpenAI in v1 SDK #766

Merged
merged 2 commits into from
Nov 16, 2023
Merged

Conversation

ashjeanbird
Copy link
Contributor

  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

#692
Updates Azure OpenAI with a note warning users that Dall-E is not supported in the v1 SDK yet.

Additional context & links

Adds important note for Azure OpenAI users.
@ashjeanbird ashjeanbird requested a review from a team as a code owner November 10, 2023 01:50
@rattrayalex
Copy link
Collaborator

I don't think this is how we'd want to phrase it; better to point to the Azure docs which document a workaround. cc @kristapratico

@StephenHodgson
Copy link

StephenHodgson commented Nov 10, 2023

@rattrayalex Azure docs which document a workaround

link?

@kristapratico
Copy link
Contributor

I don't think this is how we'd want to phrase it; better to point to the Azure docs which document a workaround. cc @kristapratico

Thanks, checking with the team.

link?

https://learn.microsoft.com/en-us/azure/ai-services/openai/how-to/migration?tabs=python%2Cdalle-fix#known-issues

 Included a link to the workaround
@ashjeanbird
Copy link
Contributor Author

Hey, @rattrayalex & @kristapratico I have added the link and the workaround to reflect the suggested doc changes. Thank you for your review.

@schnerd schnerd merged commit 23d03a1 into openai:main Nov 16, 2023
1 check passed
@rattrayalex
Copy link
Collaborator

rattrayalex commented Nov 16, 2023 via email

@schnerd schnerd mentioned this pull request Nov 16, 2023
1 task
@schnerd
Copy link
Collaborator

schnerd commented Nov 16, 2023

Ah my bad – removing here https://github.com/openai/openai-python/pull/830/files

@StephenHodgson
Copy link

It's probably fine to keep for now until they actually get it in

megamanics pushed a commit to devops-testbed/openai-python that referenced this pull request Aug 14, 2024
* Adds DALL-E note to Azure OpenAI in v1 SDK

Adds important note for Azure OpenAI users.

* Updates DALL-E docs to include workaround

 Included a link to the workaround
cgayapr pushed a commit to cgayapr/openai-python that referenced this pull request Dec 14, 2024
* Adds DALL-E note to Azure OpenAI in v1 SDK

Adds important note for Azure OpenAI users.

* Updates DALL-E docs to include workaround

 Included a link to the workaround
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.

5 participants