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 extract nodes from text prompt #106

Merged
merged 1 commit into from
Sep 11, 2024
Merged

Conversation

prasmussen15
Copy link
Collaborator

@prasmussen15 prasmussen15 commented Sep 11, 2024

🚀 This description was created by Ellipsis for commit 8c2d86e

Summary:

This PR adds functionality to extract entity nodes from text prompts by introducing extract_text in graphiti_core/prompts/extract_nodes.py and updating extract_nodes in graphiti_core/utils/maintenance/node_operations.py.

Key points:

  • Added extract_text function in graphiti_core/prompts/extract_nodes.py for extracting nodes from text prompts.
  • Updated Versions dictionary in graphiti_core/prompts/extract_nodes.py to include extract_text.
  • Added extract_text_nodes function in graphiti_core/utils/maintenance/node_operations.py to handle text-based node extraction.
  • Modified extract_nodes function in graphiti_core/utils/maintenance/node_operations.py to call extract_text_nodes for EpisodeType.text.

Generated with ❤️ by ellipsis.dev

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 8c2d86e in 11 seconds

More details
  • Looked at 113 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. graphiti_core/prompts/extract_nodes.py:161
  • Draft comment:
    The guideline numbering starts from 2 instead of 1. It should be corrected for clarity.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The guideline numbering in the extract_text function is incorrect. It starts from 2 instead of 1, which can cause confusion.

Workflow ID: wflow_UcxUTed0X7XM11PO


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@prasmussen15 prasmussen15 merged commit 4122d35 into main Sep 11, 2024
6 checks passed
@prasmussen15 prasmussen15 deleted the extract-text-prompt branch September 11, 2024 16:06
@github-actions github-actions bot locked and limited conversation to collaborators Sep 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants