-
Notifications
You must be signed in to change notification settings - Fork 6.7k
Add mem0 Memory Implementation #6510
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
base: main
Are you sure you want to change the base?
Conversation
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
@microsoft-github-policy-service agree |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! Could you add unit tests so we can ensure the correctness of the features?
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6510 +/- ##
==========================================
- Coverage 79.29% 78.63% -0.67%
==========================================
Files 225 226 +1
Lines 16679 16812 +133
==========================================
- Hits 13226 13220 -6
- Misses 3453 3592 +139
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Thanks for replying 🚀 Unit test added: https://github.com/alpha-xone/autogen/actions/runs/14992498495 |
Thanks for adding the tests. Can we use the open source verison of mem0? I think it can run entirely locally. |
Yes we can. I'm running it locally with the combination of neo4j for graph storage + chromadb for vector db + SQLite for historical db. All we need to do is setup the local environment and pass a dict of configurations of these in. But for unit testing, I'm not sure how to make these environments ready to test. |
For SQLite and Chroma DB, I think you can instantiate directly as test fixture, right? See example on For Neo4j, there should be github actions that can set this up in CI. For local integration test, we can skip it if no neo4j server is running. Besides all these above, I thought mem0's can run independently as in-memory python code? That's my impression reading their documentation. https://docs.mem0.ai/open-source/python-quickstart#basic |
The
I can only test locally with the same config in the |
Why are these changes needed?
These changes are needed to expand AutoGen's memory capabilities with a robust, production-ready integration with Mem0.ai.
This PR adds a new memory component for AutoGen that integrates with Mem0.ai, providing a robust memory solution that supports both cloud and local backends. The Mem0Memory class enables agents to store and retrieve information persistently across conversation sessions.
Key Features
Related issue number
Checks