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

docs: elasticsearch retriever #18965

Merged
merged 2 commits into from Mar 12, 2024
Merged

Conversation

maxjakob
Copy link
Contributor

@maxjakob maxjakob commented Mar 12, 2024

Add documentation notebook for ElasticsearchRetriever.

Dependencies

  • Release new langchain-elasticsearch version 0.2.0 that includes ElasticsearchRetriever

@efriis efriis added the partner label Mar 12, 2024
@efriis efriis self-assigned this Mar 12, 2024
Copy link

vercel bot commented Mar 12, 2024

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 Mar 12, 2024 11:10am

@maxjakob maxjakob marked this pull request as ready for review March 12, 2024 11:08
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. Ɑ: retriever Related to retriever module 🔌: elasticsearch Primarily related to elastic/elasticsearch integrations 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder labels Mar 12, 2024
"from langchain.embeddings import DeterministicFakeEmbedding\n",
"from langchain_core.documents import Document\n",
"from langchain_core.embeddings import Embeddings\n",
"from langchain_elasticsearch import ElasticsearchRetriever"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@efriis for this import to work we have to release a new version of the langchain-elasticsearch package that includes the class. How do we do this? Do I usually just ask you to do it?

@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Mar 12, 2024
@efriis efriis merged commit 911ccf9 into langchain-ai:master Mar 12, 2024
22 checks passed
bechbd pushed a commit to bechbd/langchain that referenced this pull request Mar 29, 2024
Add documentation notebook for `ElasticsearchRetriever`.

## Dependencies
- [ ] Release new `langchain-elasticsearch` version 0.2.0 that includes
`ElasticsearchRetriever`
gkorland pushed a commit to FalkorDB/langchain that referenced this pull request Mar 30, 2024
Add documentation notebook for `ElasticsearchRetriever`.

## Dependencies
- [ ] Release new `langchain-elasticsearch` version 0.2.0 that includes
`ElasticsearchRetriever`
hinthornw pushed a commit that referenced this pull request Apr 26, 2024
Add documentation notebook for `ElasticsearchRetriever`.

## Dependencies
- [ ] Release new `langchain-elasticsearch` version 0.2.0 that includes
`ElasticsearchRetriever`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder 🔌: elasticsearch Primarily related to elastic/elasticsearch integrations lgtm PR looks good. Use to confirm that a PR is ready for merging. partner Ɑ: retriever Related to retriever module size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants