-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Added Embeddings Option on custom Triplets #10629
Added Embeddings Option on custom Triplets #10629
Conversation
storage/default__vector_store.json
Outdated
@@ -0,0 +1 @@ | |||
{ "embedding_dict": {}, "text_id_to_ref_doc_id": {}, "metadata_dict": {} } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need to commit those json files?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, they were created by make format and make lint for some reason, added automatically, so I did not touch them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we remove them
I don't think they are needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok sure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done @hatianzhang , please check
@abhiram1809 since we just upgrade to 0.10.* |
Thanks @hatianzhang , I got a bit busy |
Description
Added an option to include embeddings for triplets that are upserted manually in the Knowledge Graph Index increasing their fetch-ability in the System
Type of Change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Suggested Checklist:
make format; make lint
to appease the lint gods