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.

Sign up for GitHub

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

Jump to bottom

Azure OpenAI Embeddings failed due to no deployment_id set. #5001

Closed
3 of 14 tasks
tot0 opened this issue May 19, 2023 · 2 comments · Fixed by #5002
Closed
3 of 14 tasks

Azure OpenAI Embeddings failed due to no deployment_id set. #5001

tot0 opened this issue May 19, 2023 · 2 comments · Fixed by #5002

Comments

@tot0
Copy link
Contributor

tot0 commented May 19, 2023

System Info

Broken by #4915

Error: Must provide an 'engine' or 'deployment_id' parameter to create a <class 'openai.api_resources.embedding.Embedding'>

I'm putting a PR out to fix this now.

Who can help?

No response

Information

  • The official example notebooks/scripts
  • My own modified scripts

Related Components

  • LLMs/Chat Models
  • Embedding Models
  • Prompts / Prompt Templates / Prompt Selectors
  • Output Parsers
  • Document Loaders
  • Vector Stores / Retrievers
  • Memory
  • Agents / Agent Executors
  • Tools / Toolkits
  • Chains
  • Callbacks/Tracing
  • Async

Reproduction

Run example notebook: https://github.com/hwchase17/langchain/blob/22d844dc0795e7e53a4cc499bf4974cb83df490d/docs/modules/models/text_embedding/examples/azureopenai.ipynb

Expected behavior

Embedding using Azure OpenAI should work.
@tot0 tot0 mentioned this issue May 19, 2023
Merged
@dev2049 dev2049 mentioned this issue May 19, 2023
Merged
dev2049 added a commit that referenced this issue May 19, 2023
@dev2049
Revert "API update: Engines -> Models (#4915)" (#5008)
9928fb2 
This reverts commit 8c28ad6.

Seems to be causing #5001
@tot0
Copy link
Contributor Author

tot0 commented May 20, 2023

Thanks @dev2049

@tot0 tot0 closed this as completed May 20, 2023
@assert6
Copy link
Contributor

assert6 commented May 22, 2023

I'm sorry about that PR 😣

@dosubot dosubot bot mentioned this issue Sep 16, 2023
Closed
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ensure deployment_id is set to provided deployment, required for Azure OpenAI. tot0/langchain
2 participants
@tot0 @assert6