Skip to content

Commit d33547a

Browse files
authored
User/dyadav/fix telemetry issue (#2541)
1 parent d77bed2 commit d33547a

File tree

7 files changed

+20
-57
lines changed

7 files changed

+20
-57
lines changed

mem0/client/main.py

+5-4
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,17 @@
55
from typing import Any, Dict, List, Optional, Union
66

77
import httpx
8-
import hashlib
98

9+
from mem0.memory.setup import get_user_id, setup_config
1010
from mem0.memory.telemetry import capture_client_event
1111

1212
logger = logging.getLogger(__name__)
1313

1414
warnings.filterwarnings("default", category=DeprecationWarning)
1515

16+
# Setup user config
17+
setup_config()
18+
1619

1720
class APIError(Exception):
1821
"""Exception raised for errors in the API."""
@@ -75,13 +78,11 @@ def __init__(
7578
self.host = host or "https://api.mem0.ai"
7679
self.org_id = org_id
7780
self.project_id = project_id
81+
self.user_id = get_user_id()
7882

7983
if not self.api_key:
8084
raise ValueError("Mem0 API Key not provided. Please provide an API Key.")
8185

82-
# Create MD5 hash of API key for user_id
83-
self.user_id = hashlib.md5(self.api_key.encode()).hexdigest()
84-
8586
self.client = httpx.Client(
8687
base_url=self.host,
8788
headers={"Authorization": f"Token {self.api_key}", "Mem0-User-ID": self.user_id},

mem0/configs/base.py

+1-4
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,9 @@
66
from mem0.embeddings.configs import EmbedderConfig
77
from mem0.graphs.configs import GraphStoreConfig
88
from mem0.llms.configs import LlmConfig
9+
from mem0.memory.setup import mem0_dir
910
from mem0.vector_stores.configs import VectorStoreConfig
1011

11-
# Set up the directory path
12-
home_dir = os.path.expanduser("~")
13-
mem0_dir = os.environ.get("MEM0_DIR") or os.path.join(home_dir, ".mem0")
14-
1512

1613
class MemoryItem(BaseModel):
1714
id: str = Field(..., description="The unique identifier for the text data")

mem0/configs/vector_stores/azure_ai_search.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ class AzureAISearchConfig(BaseModel):
77
collection_name: str = Field("mem0", description="Name of the collection")
88
service_name: str = Field(None, description="Azure AI Search service name")
99
api_key: str = Field(None, description="API key for the Azure AI Search service")
10-
embedding_model_dims: int = Field(1536, description="Dimension of the embedding vector")
10+
embedding_model_dims: int = Field(None, description="Dimension of the embedding vector")
1111
compression_type: Optional[str] = Field(
1212
None, description="Type of vector compression to use. Options: 'scalar', 'binary', or None"
1313
)

mem0/memory/main.py

+1-11
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import hashlib
44
import json
55
import logging
6-
import os
76
import uuid
87
import warnings
98
from datetime import datetime
@@ -19,7 +18,7 @@
1918
get_update_memory_messages,
2019
)
2120
from mem0.memory.base import MemoryBase
22-
from mem0.memory.setup import mem0_dir, setup_config
21+
from mem0.memory.setup import setup_config
2322
from mem0.memory.storage import SQLiteManager
2423
from mem0.memory.telemetry import capture_event
2524
from mem0.memory.utils import (
@@ -63,15 +62,6 @@ def __init__(self, config: MemoryConfig = MemoryConfig()):
6362
self.graph = MemoryGraph(self.config)
6463
self.enable_graph = True
6564

66-
self.config.vector_store.config.collection_name = "mem0_migrations"
67-
if self.config.vector_store.provider in ["faiss", "qdrant"]:
68-
provider_path = f"migrations_{self.config.vector_store.provider}"
69-
self.config.vector_store.config.path = os.path.join(mem0_dir, provider_path)
70-
os.makedirs(self.config.vector_store.config.path, exist_ok=True)
71-
72-
self._telemetry_vector_store = VectorStoreFactory.create(
73-
self.config.vector_store.provider, self.config.vector_store.config
74-
)
7565
capture_event("mem0.init", self)
7666

7767
@classmethod

mem0/memory/setup.py

-27
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import uuid
44

55
# Set up the directory path
6-
VECTOR_ID = str(uuid.uuid4())
76
home_dir = os.path.expanduser("~")
87
mem0_dir = os.environ.get("MEM0_DIR") or os.path.join(home_dir, ".mem0")
98
os.makedirs(mem0_dir, exist_ok=True)
@@ -30,29 +29,3 @@ def get_user_id():
3029
return user_id
3130
except Exception:
3231
return "anonymous_user"
33-
34-
35-
def get_or_create_user_id(vector_store):
36-
"""Store user_id in vector store and return it."""
37-
user_id = get_user_id()
38-
39-
# Try to get existing user_id from vector store
40-
try:
41-
existing = vector_store.get(vector_id=VECTOR_ID)
42-
if existing and hasattr(existing, "payload") and existing.payload and "user_id" in existing.payload:
43-
return existing.payload["user_id"]
44-
except:
45-
pass
46-
47-
# If we get here, we need to insert the user_id
48-
try:
49-
dims = getattr(vector_store, "embedding_model_dims", 1)
50-
vector_store.insert(
51-
vectors=[[0.0] * dims],
52-
payloads=[{"user_id": user_id, "type": "user_identity"}],
53-
ids=[VECTOR_ID]
54-
)
55-
except:
56-
pass
57-
58-
return user_id

mem0/memory/telemetry.py

+11-9
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
from posthog import Posthog
77

88
import mem0
9-
from mem0.memory.setup import get_or_create_user_id
9+
from mem0.memory.setup import get_user_id, setup_config
1010

1111
MEM0_TELEMETRY = os.environ.get("MEM0_TELEMETRY", "True")
1212

@@ -21,11 +21,11 @@
2121

2222

2323
class AnonymousTelemetry:
24-
def __init__(self, project_api_key, host, vector_store=None):
24+
def __init__(self, project_api_key, host):
2525
self.posthog = Posthog(project_api_key=project_api_key, host=host)
26-
27-
self.user_id = get_or_create_user_id(vector_store)
28-
26+
# Call setup config to ensure that the user_id is generated
27+
setup_config()
28+
self.user_id = get_user_id()
2929
if not MEM0_TELEMETRY:
3030
self.posthog.disabled = True
3131

@@ -50,12 +50,14 @@ def close(self):
5050
self.posthog.shutdown()
5151

5252

53-
def capture_event(event_name, memory_instance, additional_data=None):
54-
global telemetry
53+
# Initialize AnonymousTelemetry
54+
telemetry = AnonymousTelemetry(
55+
project_api_key="phc_hgJkUVJFYtmaJqrvf6CYN67TIQ8yhXAkWzUn9AMU4yX",
56+
host="https://us.i.posthog.com",
57+
)
5558

56-
# For OSS, we use the telemetry vector store to store the user_id
57-
telemetry = AnonymousTelemetry(project_api_key="phc_hgJkUVJFYtmaJqrvf6CYN67TIQ8yhXAkWzUn9AMU4yX", host="https://us.i.posthog.com", vector_store=memory_instance._telemetry_vector_store if hasattr(memory_instance, "_telemetry_vector_store") else None)
5859

60+
def capture_event(event_name, memory_instance, additional_data=None):
5961
event_data = {
6062
"collection": memory_instance.collection_name,
6163
"vector_size": memory_instance.embedding_model.config.embedding_dims,

pyproject.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[tool.poetry]
22
name = "mem0ai"
3-
version = "0.1.89"
3+
version = "0.1.90"
44
description = "Long-term memory for AI Agents"
55
authors = ["Mem0 <[email protected]>"]
66
exclude = [

0 commit comments

Comments
 (0)