Skip to content

community[minor]: Add keybert-based link extractor #24311

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

Merged
merged 9 commits into from
Jul 19, 2024

Conversation

bjchambers
Copy link
Contributor

  • Description: Add a KeybertLinkExtractor for graph vectorstores. This allows extracting links from keywords in a Document and linking nodes that have common keywords.
  • Issue: None
  • Dependencies: None.

This allows extracting links from keywords in a Document and linking
nodes that have common keywords.
Copy link

vercel bot commented Jul 16, 2024

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Jul 19, 2024 3:51pm

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. community Related to langchain-community Ɑ: vector store Related to vector store module 🤖:improvement Medium size change to existing code to handle new use-cases labels Jul 16, 2024
@eyurtsev eyurtsev self-assigned this Jul 16, 2024
@eyurtsev eyurtsev changed the title community: Add keybert-based link extractor community[minor]: Add keybert-based link extractor Jul 19, 2024
@eyurtsev eyurtsev enabled auto-merge (squash) July 19, 2024 13:53
auto-merge was automatically disabled July 19, 2024 15:17

Head branch was pushed to by a user without write access

LinkExtractor,
)

# TypeAlias is not available in Python 2.9, we can't use that or the newer `type`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Python 3.8 is minimum

embedding_model: str = "all-MiniLM-L6-v2",
extract_keywords_kwargs: Optional[Dict[str, Any]] = None,
):
"""Extract keywords using Keybert.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you add in code examples and a link to keybert docs? (Same comment as in the other PRs)

@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Jul 19, 2024
@eyurtsev eyurtsev merged commit 3691701 into langchain-ai:master Jul 19, 2024
43 checks passed
olgamurraft pushed a commit to olgamurraft/langchain that referenced this pull request Aug 16, 2024
- **Description:** Add a `KeybertLinkExtractor` for graph vectorstores.
This allows extracting links from keywords in a Document and linking
nodes that have common keywords.
- **Issue:** None
- **Dependencies:** None.

---------

Co-authored-by: Eugene Yurtsev <[email protected]>
Co-authored-by: ccurme <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Related to langchain-community 🤖:improvement Medium size change to existing code to handle new use-cases lgtm PR looks good. Use to confirm that a PR is ready for merging. size:L This PR changes 100-499 lines, ignoring generated files. Ɑ: vector store Related to vector store module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants