8000 Integrate with Mem0 OSS by Dev-Khant · Pull Request #1678 · pipecat-ai/pipecat · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Integrate with Mem0 OSS #1678

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Apr 28, 2025
Merged

Integrate with Mem0 OSS #1678

merged 3 commits into from
Apr 28, 2025

Conversation

Dev-Khant
Copy link
Contributor
@Dev-Khant Dev-Khant commented Apr 28, 2025 •
edited
Loading

Integrating Mem0 OSS version here.

Changes:

  • Along with Mem0 platform now this PR enables the support for Mem0 OSS package.
  • Change in Mem0 example to showcase both functionality
  • Bumped version of Mem0 to latest: 0.1.94

@markbackman
Copy link
Contributor
markbackman commented Apr 28, 2025 •
edited
Loading

This is great @Dev-Khant! Would you mind adding a changelog entry for this too?

Can you also run the pre-commit hook or set up Ruff for linting?

@codecov Codecov
Copy link
codecov bot commented Apr 28, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pipecat/services/mem0/memory.py 0.00% 15 Missing ⚠️
Files with missing lines Coverage Δ
src/pipecat/services/mem0/memory.py 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@markbackman markbackman self-requested a review April 28, 2025 12:41
@Dev-Khant
Copy link
Contributor Author

Hey @markbackman Thanks for reviewing the PR, I have updated changelog and also formatted the files.

@markbackman
Copy link
Contributor

@Dev-Khant two more linting fixes needed. Both need:

from mem0 import Memory, MemoryClient

@Dev-Khant
Copy link
Contributor Author

@markbackman I think it's fixed. Precommit checks are passing locally.

markbackman
markbackman approved these changes Apr 28, 2025
View reviewed changes
Copy link
Contributor
@markbackman markbackman left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the fixes and the contribution 🙇

@markbackman markbackman merged commit 3d05a74 into pipecat-ai:main Apr 28, 2025
5 of 6 checks passed
@Dev-Khant Dev-Khant mentioned this pull request Apr 28, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markbackman markbackman markbackman approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Dev-Khant @markbackman
0