-
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
epic: Index #2
Comments
@amrhssn had a meeting with Farhoud and Ramin about interfaces. the conclusion was to have the interfaces based on the usage on the Digest and Retrieve side.
feel free to make fun of it |
@mehdibalouchi I did some initial experiments, and as expected, the naive way of returning the top similarity score between the query and data embeddings doesn't work well. The first interface for the Retrieve module is good but I also need all other attributes for pre/post-processing the initial results. About the second interface, I'd say we wait and put off the extra engineering after we're happy with one end-to-end cycle of the app. We need to spend time on the enrichment and the indexing process itself. Let me know when you're free to talk 🙌🏻 |
A module that takes a graph and index it in a database.
The text was updated successfully, but these errors were encountered: