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

Still facing the AZURE embedding issue #176

Closed
shagu2021 opened this issue Oct 29, 2023 · 3 comments · Fixed by #192
Closed

Still facing the AZURE embedding issue #176

shagu2021 opened this issue Oct 29, 2023 · 3 comments · Fixed by #192

Comments

@shagu2021
Copy link

git log
commit 2911989

i still see the auth error when i use AZURE and embeddings
AuthenticationError: No API key provided. You can set your API key in code using 'openai.api_key = ', or you can set the environment variable OPENAI_API_KEY=). If your API
key is stored in a file, you can point the openai module at it with 'openai.api_key_path = '. You can generate API keys in the OpenAI web interface. See

following is my command

image

MemGPT > memgpt --archival-storage-files-compute-embeddings="*.pdf" --persona=memgpt_doc --human=basic --use-azure-openai

@s-spillias
Copy link

similar issue: I have defined the new 'AZURE_OPENAI_EMBEDDING_DEPLOYMENT' variable using our deployment name for embedgginds. But it looks like it is trying to reach this through the gpt4 endpoint, whereas, we have a different endpoint for embeddings.

The specific error is:
InvalidRequestError: The API deployment for this resource does not exist. If you created the deployment within the last
5 minutes, please wait a moment and try again.

@cpacker
Copy link
Collaborator

cpacker commented Oct 30, 2023

@vivi I think this is because of a typo here: https://github.com/cpacker/MemGPT/blob/main/memgpt/openai_tools.py#L150

Patching now, will link the PR.

@vivi
Copy link
Contributor

vivi commented Oct 30, 2023

Define AZURE_OPENAI_EMBEDDINGS_DEPLOYMENT. (EMBEDDINGS -- see README).

The patch we pushed makes everything consistent (to use AZURE_OPENAI_EMBEDDINGS_DEPLOYMENT) now.

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 a pull request may close this issue.

4 participants