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

community: make elastic_search_bm25 retriever directly index documents, accept user-input analyzer for flexible full-text search #29702

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

bluearrow98
Copy link

Description: Extend full-text search capability of elastic_search_bm25 retriever by accepting additional parameters such as analyzer type, client kwargs and possibility to directly index documents.

docs: Improved existing documentation.

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Feb 9, 2025
Copy link

vercel bot commented Feb 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 13, 2025 8:29pm

@ccurme
Copy link
Collaborator

ccurme commented Feb 12, 2025

This PR is currently modifying 2,325 files, could you please remove the irrelevant changes?

@ccurme ccurme self-assigned this Feb 12, 2025
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Feb 13, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Feb 13, 2025
@bluearrow98
Copy link
Author

Thanks @ccurme. Just did that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Related to langchain-community Ɑ: retriever Related to retriever module size:L This PR changes 100-499 lines, ignoring generated files.
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

2 participants