Updates to make this work properly with the latest pinecone and openai#1
Updates to make this work properly with the latest pinecone and openai#1eharvey71 wants to merge 2 commits intoashnkumar:mainfrom
Conversation
|
hey @eharvey71 is it working fine for you after changes? |
It is! I just tested again today against different sites and all appears to be working.
|
|
awesome! I have azureopenai api key and I am wondering how I can use it instead of openai api key and if it will work or not. would love to know if you are aware of it. |
|
Ah, that would be interesting to try. I don't have Azure credentials,
though. Good luck giving that a shot!
…On Thu, Jan 25, 2024 at 2:21 AM vinayak gavariya ***@***.***> wrote:
awesome!
I have azureopenai api key and I am wondering how I can use it instead of
openai api key and if it will work or not.
would love to know if you are aware of it.
—
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAH6LD3QH5ZDADHRPVGJNNTYQIBWNAVCNFSM6AAAAABCCKPKDKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBZGQ4DMMRUGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
Eric Harvey
804-647-7522
|
app/services/pinecone_service.py
Outdated
| @@ -1,19 +1,23 @@ | |||
| import pinecone | |||
| from pinecome import Pinecone, ServerlessSpec | |||
There was a problem hiding this comment.
Should this be pinecone instead of pinecome
|
Just tested again and all seems well with my pull request. Can someone review? |



Made some updates, mostly to the pinecone service module. This wasn't working for me and I've corrected the following: