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

Feat: Adding Mem0 Integration #4339

Merged
merged 9 commits into from
Nov 11, 2024
Merged

Conversation

joaoguilhermeS
Copy link
Collaborator

@joaoguilhermeS joaoguilhermeS commented Oct 31, 2024

This pull request introduces integration with Mem0 memory storage, supporting both self-hosted and cloud versions. The following components have been added:

  • Mem0 Chat Memory: Retrieves and stores chat messages using Mem0 memory.
  • Mem0 Chat Memory Update: Updates messages in Mem0 memory storage.
  • Mem0 Chat Memory Delete: Deletes messages from Mem0 memory storage.

image

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels Oct 31, 2024
@edwinjosechittilappilly
Copy link
Collaborator

@NadirJ @rodrigosnader Should we move this component to integrations/ bundles?

Copy link
Collaborator

@edwinjosechittilappilly edwinjosechittilappilly left a comment

Choose a reason for hiding this comment

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

Would it be better in bundles or integrations?

update uv for memo
update and delete mem0 removed as per suggestion
@edwinjosechittilappilly
Copy link
Collaborator

To merge the PR, I am removing the Update and deleting Mem0Component. Please connect with cc: @carlosrcoelho and @rodrigosnader , for further info. We could add it in the future if required.

Copy link
Collaborator

@edwinjosechittilappilly edwinjosechittilappilly left a comment

Choose a reason for hiding this comment

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

LGTM @joaoguilhermeS can you test the mem0 once again if possible.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 8, 2024
Copy link

codspeed-hq bot commented Nov 8, 2024

CodSpeed Performance Report

Merging #4339 will degrade performances by 12.66%

Comparing jg/integration/mem0 (e98ffe6) with main (76c91de)

Summary

❌ 1 regressions
✅ 14 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark main jg/integration/mem0 Change
test_create_starter_projects 402.7 ms 461.1 ms -12.66%

@edwinjosechittilappilly edwinjosechittilappilly merged commit 5a6c556 into main Nov 11, 2024
29 checks passed
@edwinjosechittilappilly edwinjosechittilappilly deleted the jg/integration/mem0 branch November 11, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants