dspy
dspy copied to clipboard
Fix bugs in pinecone integration, add support for using cohere embed with pinecone and create basic example notebook
@CShorten , @okhat this should resolve #322. I removed the _init_pinecone function because I believe it shouldn't be the responsibility of the PineconeRM class to create an index. Let me know what you think.
Thank you for the detailed descriptions of the issue and the fix. I did a cursory look at it and it looks good to me.
We need to improve this document for other RMs, but if you could add the entry for pinecone, it would be great.
-
docs/retrieval_models_client.md
Hey @insop , I added an entry for pinecone in the docs
Hey @insop , I added an entry for pinecone in the docs
Thank you so much, looks good to me.
Hi @DanielUH2019 , could you merge conflicts locally and commit again? seems like I'm unable to do it within the local command line here but the changes are good to merge!
@DanielUH2019 any updates on addressing the conflicts? facing the same issue with Pinecone
AttributeError: init is no longer a top-level attribute of the pinecone package.