-
Notifications
You must be signed in to change notification settings - Fork 670
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
[FEATURE] Is milvus support final? #114
Comments
Hi @AbdullahGheith we've added support for Milvus, but there are still some problems found in the last minute before release and we decided to not announce it before we fix them. Please share your implementation, it is interesting to see what is your approach. It seems that you are talking about "metadata filtering" If I understood correctly. This is not possible yet in our implementation, but we are going to add it soon, as this is an important feature. So if you already have this, would be awesome to get your contribution! Thanks! |
Heres my implementation. It works, and i use it currently:
|
@AbdullahGheith thanks a lot! I will incorporate some of your code into our current implementation. |
@AbdullahGheith we've updated Milvus support, please check 0.23.0: https://github.com/langchain4j/langchain4j-examples/blob/main/milvus-example/src/main/java/MilvusEmbeddingStoreExample.java |
I've had created my own Milvus implementation but i havent shared it because i've kind of did it in a different way.
I see now that milvus has been added and maybe i should use the official one. (But i am in doubt, because its not mentioned in the changelog + i dont know if it can do what i wanna do)
The main difference is that in mine, i've added an additional id to my vectors, which is my document id.
So can save these objects seperately:
DocumentId: Book1
Text: The sky is blue
DocumentId: Book2
Text: The sky is red
Then i can do:
findRelevant("Book1", "What color is the sky", 1);
Then my milvus search will only search in all the "Book1" vectors.
I dont know if this is possible with current EmbeddingStore interface, but i havent figured a way to do it, so i made it myself.
Is this something possible to do or should i share my code?
The text was updated successfully, but these errors were encountered: