Skip to content

[api][runtime] Support Mem0 based Long-Term Memory in Python. - #629

Merged
wenjin272 merged 3 commits into
apache:mainfrom
wenjin272:mem0-python
Apr 29, 2026
Merged

wenjin272 merged 3 commits into
apache:mainfrom
wenjin272:mem0-python

Conversation

@wenjin272

@wenjin272 wenjin272 commented Apr 14, 2026 •

Copy link
Copy Markdown
Contributor

Linked issue: #618 and #620

Purpose of change

  1. Remove VectorStoreLongTermMemory
  2. Introduce Mem0LongTermMemory

Tests

ut & e2e

API

Documentation

  • doc-needed
  • doc-not-needed
  • doc-included

@github-actions github-actions Bot added doc-needed Your PR changes impact docs. fixVersion/0.3.0 The feature or bug should be implemented/fixed in the 0.3.0 version. priority/major Default priority of the PR or issue. and removed doc-needed Your PR changes impact docs. labels Apr 14, 2026
@wenjin272
wenjin272 force-pushed the mem0-python branch 4 times, most recently from c2b6b6e to a7d05b0 Compare April 16, 2026 07:06
@wenjin272
wenjin272 marked this pull request as draft April 16, 2026 07:57
@wenjin272
wenjin272 marked this pull request as ready for review April 16, 2026 13:51
VECTOR_STORE = ConfigOption(
key="long-term-memory.vector-store",
config_type=dict,
default=None,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  1. I'd suggest the names long-term-memory.mem0.xxx, in case we introduce more backends in future.
  2. Having a dict type config option can be troublesome. Why is the vector store configured differently compared to chat and embedding model?

job_id=job_identifier,
chat_model_name=chat_model_name,
embedding_model_name=embedding_model_name,
vector_store_config=vector_store_config,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The initialization of ltm-beckend might worth a separate function.

@wenjin272
wenjin272 force-pushed the mem0-python branch 2 times, most recently from 228b6ff to 88607d8 Compare April 18, 2026 16:11
@wenjin272
wenjin272 force-pushed the mem0-python branch 3 times, most recently from 5b806e8 to 1b5e31b Compare April 19, 2026 08:40
wenjin272 and others added 3 commits April 21, 2026 17:36
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@xintongsong xintongsong left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@wenjin272
wenjin272 merged commit 3951823 into apache:main Apr 29, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-needed Your PR changes impact docs. fixVersion/0.3.0 The feature or bug should be implemented/fixed in the 0.3.0 version. priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants