Insop comments

Results 100 comments of


                                            Insop

Refactored ChromadbRM to use generic EmbeddingFunction

> Yes, `model_arg` was specifically introduced to maintain compatibility with the Azure OpenAI API. However, the current implementation supports OpenAI embeddings. You can find the details on how to define...

fixes _get_embeddings for ChromaDB

@lukaskuhn-lku, it seems like https://github.com/stanfordnlp/dspy/pull/400 update has wider scope changes. Maybe this PR might not be necessary if the newer one (400) is reviewed and merged.

fix react.py #276

LGTM for the PR, it'd be great to take the changes from @smwitkowski .

Fix bugs in pinecone integration, add support for using cohere embed with pinecone and create basic example notebook

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...

Fix bugs in pinecone integration, add support for using cohere embed with pinecone and create basic example notebook

> Hey @insop , I added an entry for pinecone in the docs Thank you so much, looks good to me.

[feat] deeplake retriever implementation

Could you also update this document? - `docs/retrieval_models_client.md`

Colab pip install issue

I ran this about a week ago on the colab, it didn't work.

fix(colbertv2): Add error handler for server reports error

Thank you for the PR. It looks good to me.

Improve checking to detect `NaN` case

PTAL, @meenchen , @RaymondWang0. Thank you.

[Bug]: "Error fetching /config.json" error

Thank you @mamoodi, `Cmd + Shift + R` did not work, but the incognito mode works. So somewhere in the browser cache is corrupted and forbid the app started correctly....