-
Notifications
You must be signed in to change notification settings - Fork 1
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
Updated notebook #3
base: main
Are you sure you want to change the base?
Conversation
Updated notebook to have metadata and top k documents returned
to remove -copy2
don't forget to set the Projects field |
Note that this is all locally stored. Default vector store created using this serialisation creates a .json file of size 3gb.
@leejaeka have a look at the checks to resolve the failing checks (LICENSE and TESTING.md). |
@leejaeka fix failing checks before submitting for review: |
llamaindex-load.ipynb notebook contains Python code to download llamaindex query engine v1 from Azure Storage Container locally. Notebook also shows how to query and retrieve metadata and data for debugging and testing.
TESTING.md is temporay. Will fill them out |
@leejaeka looks like @k-allagbe had made changes separately that now conflict with yours. rebase your branch on the main branch to resolve the conflict. |
@leejaeka you've used a merge strategy instead of rebase. we prefer rebase to keep a clean history on the main branch. see: "Handling divergent branches" |
Created llamaindex-test.ipynb that returns test accuracy. Llamaindex updated some packages so some changes to importing codes.
Openai updated importing codes and packages were not being located correctly
load api key added
@leejaeka please address the failing checks: https://github.com/ai-cfia/llamaindex-db/actions/runs/8190566290/job/22397803331?pr=3 |
Updated notebook to have metadata and top k documents returned. Please read the comments in the notebook for more information.
Currently it processes .csv file and only connects to postgresql db server.