Skip to content

Added Mem0 Instrumentation#1027

Merged
dot-agi merged 45 commits into
mainfrom
mem0_instrumentation
Jun 13, 2025
Merged

Added Mem0 Instrumentation#1027
dot-agi merged 45 commits into
mainfrom
mem0_instrumentation

Conversation

@fenilfaldu

@fenilfaldu fenilfaldu commented May 31, 2025

Copy link
Copy Markdown
Contributor

📥 Pull Request

closes #995

📘 Description
Add Mem0 and Concurrent Futures Instrumentation(added here for testing)

  • Introduced instrumentation for the Mem0 memory management system, capturing telemetry data for memory operations across 4 core classes (Memory, MemoryClient, AsyncMemoryClient, and AsyncMemory).
  • Implemented comprehensive wrappers for 8 key Mem0 operations (add, search, get, get_all, delete, delete_all, update, history) with proper attribute extraction and telemetry capture.
  • Added logging for better traceability during instrumentation and uninstrumentation processes.

This update enhances the overall observability of memory operations and concurrent task execution within the AgentOps framework, providing full coverage for both synchronous and asynchronous Mem0 workflows.

🧪 Testing
Screenshot 2025-06-01 at 4 20 01 AM

Also submitted pr to the mem0 repository for documentation updates.

@fenilfaldu
fenilfaldu requested review from braelyn-ai and tcdent May 31, 2025 22:52
@codecov

codecov Bot commented May 31, 2025

Copy link
Copy Markdown

Codecov Report

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

Files with missing lines Patch % Lines
agentops/instrumentation/mem0/memory.py 0.00% 196 Missing ⚠️
agentops/instrumentation/mem0/common.py 0.00% 192 Missing ⚠️
agentops/instrumentation/mem0/instrumentor.py 0.00% 43 Missing ⚠️
agentops/instrumentation/mem0/__init__.py 0.00% 14 Missing ⚠️

📢 Thoughts on this report? Let us know!

@dot-agi

dot-agi commented Jun 1, 2025

Copy link
Copy Markdown
Member

Check linting

@dot-agi

dot-agi commented Jun 1, 2025

Copy link
Copy Markdown
Member

@fenilfaldu can you add an example and a doc for this?

tcdent
tcdent previously requested changes Jun 6, 2025

@tcdent tcdent 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.

Very thorough! Just a couple notes about naming and this looks good to me.

Comment thread agentops/instrumentation/concurrent_futures/instrumentation.py Outdated
Comment thread agentops/instrumentation/mem0/common.py Outdated
Comment thread agentops/instrumentation/mem0/common.py
@areibman
areibman requested a review from dot-agi June 11, 2025 15:57

@dot-agi dot-agi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Re: #1018 please separate the code for Mem0 instrumentation and the concurrent.futures use.

Both are related but belong in a different PR as they are both features and have independent use as well.

@areibman
areibman removed request for braelyn-ai and tcdent June 13, 2025 15:48

@dot-agi dot-agi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good job @fenilfaldu!

@dot-agi
dot-agi enabled auto-merge (squash) June 13, 2025 17:50
@dot-agi
dot-agi dismissed tcdent’s stale review June 13, 2025 17:51

I am sorry :)

@dot-agi
dot-agi merged commit 3c6b2f4 into main Jun 13, 2025
9 of 10 checks passed
@dot-agi
dot-agi deleted the mem0_instrumentation branch June 13, 2025 17:51
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.

[Feature]: Integrate Mem0 memory interface

4 participants