-
Notifications
You must be signed in to change notification settings - Fork 145
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
Request to Update OpenAI Implementation to Support New API Schema #71
Request to Update OpenAI Implementation to Support New API Schema #71
Comments
I was planning to implement text-embedding-3 support this weekend, so I’ll take a look at this too. |
Update OpenAI API Call freedmand#71
@nikolas-rauscher I have a new commit to close this issue with additional support for the new embedding model, but I can't seem to figure out why the web UI didn't start correctly. Can you see if you have the same issue on your device? Any input is appreciated. |
I had the same problem after cloning the repo and then applying the patch. I was able to work around it by installing a clean copy of Semantra v. 0.1.10 with pipx and then manually replacing models.py with the updated version in its venv sub-directory. This is on OSX v14.5 with openai 1.30.5, installed using Python 3.11.9. I'm not a developer, so please let me know if I can provide more/better details. And thanks so much for creating Semantra! |
I will give this a try and report back. I'm not the author of this app tho! Credits to @freedmand |
I have encountered an issue related to the OpenAI model. The problem arises because the current implementation relies on the outdated OpenAI API schema, which is not supported in versions of the openai library greater than 1.0.0.
The recent versions of the OpenAI API have deprecated the
openai.Embedding
interface, which the code currently relies on. This results in the following error when attempting to use the feature:Are there plans to update this?
The text was updated successfully, but these errors were encountered: