openai-cookbook icon indicating copy to clipboard operation
openai-cookbook copied to clipboard

Zilliz integration and Milvus bugfixes

Open filip-halt opened this issue 2 years ago • 2 comments
trafficstars

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.

filip-halt avatar Mar 21 '23 17:03 filip-halt

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.

ted-at-openai avatar Mar 22 '23 19:03 ted-at-openai

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.

filip-halt avatar Mar 22 '23 23:03 filip-halt

Is there anything that needs to be changed to get this merged? @ted-at-openai

filip-halt avatar Mar 28 '23 18:03 filip-halt

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.

ted-at-openai avatar Mar 28 '23 22:03 ted-at-openai