Open sagarvadodaria opened 2 weeks ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
langchain | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | May 20, 2024 3:38pm |
@baskaryan / @efriis / @eyurtsev , just wanted to bring this PR to your attention. This adds a new chunk_window_retrieval_search for Qdrant which is based on sentence window.
@baskaryan / @efriis / @eyurtsev , just wanted to bring this PR to your attention. This adds a new chunk_window_retrieval_search for Qdrant which is based on sentence window.
@baskaryan / @efriis / @eyurtsev
Hello again, just wanted to check if we can hear your thoughts about this, so that we can plan accordingly. Do you agree with this approach or this has no chance. Either way, it would help us decide our design. At the bare minimum we should add the support for adding Integers as PointIds in Qdrant. At this point, Qdrant.fromtexts(..) method only accepts List of String. Any reason not to accept List of Integers as well since Qdrant does support that?
[ ] PR message:
[ ] Add tests and docs:
docs/docs/integrations/vectorstores/qdrant.ipynb
) showing its use.[ ] Lint and test:
make format
==> No errorsmake lint
==> No errorsmake test
===> No errorsIf no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, hwchase17.