-
Notifications
You must be signed in to change notification settings - Fork 11.3k
Zilliz integration and Milvus bugfixes #259
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
Conversation
Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
Makes sense. Personally I prefer having the outputs pre-computed. That way someone can see the outputs of the example code without necessarily having to run it themselves (e.g., if they're on gibhub.com, or if they don't want to go through the trouble of setting up a vector database but do want to see what it might look like). I do recognize it makes the diffs harder to analyze. |
It might be worth having a standard output near the end that shows exactly what should be returned. The different sections just show how to get that input using each vector store. |
Is there anything that needs to be changed to get this merged? @ted-at-openai |
Merged! Thanks for your patience. I think I'm going to split up the single notebook into one for each vector DB, to help keep things manageable. Hopefully can get that done tomorrow. |
* integrating zilliz and fixing milvus bugs Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * Adding Milvus and Zilliz notebooks Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * cleaned outputs of shared notebook Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> --------- Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
* integrating zilliz and fixing milvus bugs Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * Adding Milvus and Zilliz notebooks Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * cleaned outputs of shared notebook Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> --------- Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
* integrating zilliz and fixing milvus bugs Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * Adding Milvus and Zilliz notebooks Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * cleaned outputs of shared notebook Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> --------- Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
* integrating zilliz and fixing milvus bugs Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * Adding Milvus and Zilliz notebooks Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * cleaned outputs of shared notebook Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> --------- Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
* integrating zilliz and fixing milvus bugs Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * Adding Milvus and Zilliz notebooks Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * cleaned outputs of shared notebook Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> --------- Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
* integrating zilliz and fixing milvus bugs Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * Adding Milvus and Zilliz notebooks Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> * cleaned outputs of shared notebook Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com> --------- Signed-off-by: Filip Haltmayer <filip.haltmayer@zilliz.com>
Included the Zilliz vector database, Milvus bugfixes, and more notebooks.
Also this merge aims to clear the notebook outputs of the shared notebook. Every pull is deleting other peoples outputs and including their own. I believe the best route is to just not have outputs in the notebook instead of everyone overwriting each others work.