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 license file #222

Merged
merged 60 commits into from
Oct 18, 2024
Merged

add license file #222

merged 60 commits into from
Oct 18, 2024

Conversation

paul-paliychuk
Copy link
Contributor

@paul-paliychuk paul-paliychuk commented Oct 18, 2024

Important

Adds LICENSE file and updates client classes, types, and configurations.

  • Behavior:
    • Adds LICENSE file.
    • Updates asyncio.sleep duration in memory.py from 5 to 20 seconds.
    • Modifies client.memory.get() call in memory.py to remove memory_type parameter.
    • Changes client.memory.search_sessions() call in memory.py to use user_id instead of session_ids.
  • Classes and Functions:
    • Adds GroupClient and AsyncGroupClient classes in group/client.py.
    • Updates MemoryClient and AsyncMemoryClient classes in memory/client.py.
    • Updates UserClient and AsyncUserClient classes in user/client.py.
  • Types:
    • Renames DocumentResponse to ApidataDocument and similar renames for related classes.
    • Adds new types EntityEdge, EntityNode, Episode, GraphSearchResults, and Group.
    • Updates Fact, Memory, Message, and Session types with new fields and parameters.
  • Misc:
    • Updates pyproject.toml to version 2.0.0-rc.1 and modifies dependencies.
    • Removes langchain related files and references.

This description was created by Ellipsis for 3f8e6bc. It will automatically update as commits are pushed.

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 5953525 in 1 minute and 8 seconds

More details
  • Looked at 5258 lines of code in 50 files
  • Skipped 2 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. .fernignore:12
  • Draft comment:
    The change here seems incorrect. The LICENSE file should not be removed from .fernignore. Instead, ensure that the LICENSE file is properly added to the repository.
  • Reason this comment was not posted:
    Comment was on unchanged code.

Workflow ID: wflow_XDn7mE3yxJLINh5k


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

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! Incremental review on 3f8e6bc in 9 seconds

More details
  • Looked at 208 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. LICENSE:201
  • Draft comment:
    Add a newline at the end of the file for better compatibility with Unix-based systems.
  • Reason this comment was not posted:
    Confidence changes required: 10%
    The license file is missing a newline at the end, which is a common best practice for text files.

Workflow ID: wflow_oOHbNWicLp6qJSEa


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

@paul-paliychuk paul-paliychuk merged commit 4edc4bb into main Oct 18, 2024
3 checks passed
@paul-paliychuk paul-paliychuk deleted the v2 branch October 18, 2024 14:31
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.

1 participant