Skip to content
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

Add elastic #155

Merged
merged 11 commits into from
Nov 14, 2024
Merged

Add elastic #155

merged 11 commits into from
Nov 14, 2024

Conversation

htahir1
Copy link
Contributor

@htahir1 htahir1 commented Nov 13, 2024

No description provided.

Copy link

dagshub bot commented Nov 13, 2024

Copy link
Contributor Author

@htahir1 htahir1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right now @wjayesh you replaced the entire code to work with elastic. can u rather make it so that it is a "switch" between postgres + elastic

llm-complete-guide/configs/dev/rag.yaml Show resolved Hide resolved
@wjayesh
Copy link
Contributor

wjayesh commented Nov 14, 2024

@htahir1 added the option to choose vector store in the create index pipeline, and the evaluation pipeline fetches the model metadata for the given version to choose what vector store to use now.

@wjayesh wjayesh force-pushed the feature/elasticsearch-llm-complete branch from 0905c1e to 62d3e01 Compare November 14, 2024 08:05
@htahir1 htahir1 merged commit cf029d1 into main Nov 14, 2024
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants