Skip to content

make sure embedding model name can be specified for openai embeddings #301

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

Closed
wants to merge 1 commit into from

Conversation

Robin-des-Bois
Copy link

No description provided.

@VinciGit00
Copy link
Collaborator

hi, have you tried if it works? on us it does not work ok

@Robin-des-Bois
Copy link
Author

Robin-des-Bois commented May 27, 2024

When I did this edit in the local package, it worked for me. I only enabled text-embedding-3-small in my openai env and hence I got an error. Once I applied the patch, it is working for me now.

on us it does not work ok

can you be more precise?

@VinciGit00
Copy link
Collaborator

give us the code of the example

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.

2 participants