Skip to content

Commit 6f67685

Browse files
committed
Fix ALL remaining query= to text= in notebooks
Fixed 6 notebooks that still had query= instead of text= in search_long_term_memory calls: - section-3-memory/01_working_memory_with_extraction_strategies.ipynb - section-3-memory/03_memory_integration.ipynb - section-3-memory/04_memory_tools.ipynb - section-4-optimizations/01_context_window_management.ipynb - section-4-optimizations/03_grounding_with_memory.ipynb - section-4-optimizations/05_crafting_data_for_llms.ipynb Also fixed WorkingMemory and MemoryMessage imports to use agent_memory_client.models
1 parent d6b36b0 commit 6f67685

File tree

7 files changed

+79
-16
lines changed

7 files changed

+79
-16
lines changed

python-recipes/context-engineering/notebooks/section-3-memory/01_working_memory_with_extraction_strategies.ipynb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,7 @@
209209
"]\n",
210210
"\n",
211211
"# Save to working memory\n",
212-
"from agent_memory_client import WorkingMemory, MemoryMessage\n",
212+
"from agent_memory_client.models import WorkingMemory, MemoryMessage\n",
213213
"\n",
214214
"# Convert messages to MemoryMessage format\n",
215215
"memory_messages = [MemoryMessage(**msg) for msg in messages]\n",
@@ -324,7 +324,7 @@
324324
"\n",
325325
"# Search for extracted memories\n",
326326
"extracted_memories = await memory_client.search_long_term_memory(\n",
327-
" query=\"preferences goals\",\n",
327+
" text=\"preferences goals\",\n",
328328
" limit=10\n",
329329
")\n",
330330
"\n",

python-recipes/context-engineering/notebooks/section-3-memory/03_memory_integration.ipynb

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@
171171
"print(\"\\n2. Searching long-term memory...\")\n",
172172
"user_query = \"Hi! I'm interested in learning about databases.\"\n",
173173
"long_term_memories = await memory_client.search_long_term_memory(\n",
174-
" query=user_query,\n",
174+
" text=user_query,\n",
175175
" limit=3\n",
176176
")\n",
177177
"print(f\" Relevant memories found: {len(long_term_memories)}\")\n",
@@ -188,7 +188,7 @@
188188
"\n",
189189
"# Step 4: Save working memory\n",
190190
"print(\"\\n4. Saving working memory...\")\n",
191-
"from agent_memory_client import WorkingMemory, MemoryMessage\n",
191+
"from agent_memory_client.models import WorkingMemory, MemoryMessage\n",
192192
"\n",
193193
"# Convert messages to MemoryMessage format\n",
194194
"memory_messages = [MemoryMessage(**msg) for msg in []\n",
@@ -244,7 +244,7 @@
244244
"print(\"\\n2. Searching long-term memory...\")\n",
245245
"user_query_2 = \"I prefer online courses and morning classes.\"\n",
246246
"long_term_memories = await memory_client.search_long_term_memory(\n",
247-
" query=user_query_2,\n",
247+
" text=user_query_2,\n",
248248
" limit=3\n",
249249
")\n",
250250
"print(f\" Relevant memories found: {len(long_term_memories)}\")\n",
@@ -280,7 +280,7 @@
280280
" {\"role\": \"assistant\", \"content\": response.content}\n",
281281
"])\n",
282282
"\n",
283-
"from agent_memory_client import WorkingMemory, MemoryMessage\n",
283+
"from agent_memory_client.models import WorkingMemory, MemoryMessage\n",
284284
"\n",
285285
"# Convert messages to MemoryMessage format\n",
286286
"memory_messages = [MemoryMessage(**msg) for msg in all_messages]\n",
@@ -326,7 +326,7 @@
326326
"# Search for extracted memories\n",
327327
"print(\"\\nSearching for extracted memories...\\n\")\n",
328328
"memories = await memory_client.search_long_term_memory(\n",
329-
" query=\"student preferences\",\n",
329+
" text=\"student preferences\",\n",
330330
" limit=5\n",
331331
")\n",
332332
"\n",
@@ -372,7 +372,7 @@
372372
"print(\"\\n2. Searching long-term memory...\")\n",
373373
"user_query_3 = \"What database courses do you recommend for me?\"\n",
374374
"long_term_memories = await memory_client.search_long_term_memory(\n",
375-
" query=user_query_3,\n",
375+
" text=user_query_3,\n",
376376
" limit=5\n",
377377
")\n",
378378
"print(f\" Relevant memories found: {len(long_term_memories)}\")\n",
@@ -402,7 +402,7 @@
402402
"\n",
403403
"# Step 4: Save working memory\n",
404404
"print(\"\\n4. Saving working memory...\")\n",
405-
"from agent_memory_client import WorkingMemory, MemoryMessage\n",
405+
"from agent_memory_client.models import WorkingMemory, MemoryMessage\n",
406406
"\n",
407407
"# Convert messages to MemoryMessage format\n",
408408
"memory_messages = [MemoryMessage(**msg) for msg in []\n",
@@ -447,7 +447,7 @@
447447
"# Check all memories about the student\n",
448448
"print(\"\\nAll memories about this student:\\n\")\n",
449449
"all_memories = await memory_client.search_long_term_memory(\n",
450-
" query=\"\", # Empty query returns all\n",
450+
" text=\"\", # Empty query returns all\n",
451451
" limit=20\n",
452452
")\n",
453453
"\n",

python-recipes/context-engineering/notebooks/section-3-memory/04_memory_tools.ipynb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,7 @@
249249
" \"\"\"\n",
250250
" try:\n",
251251
" memories = await memory_client.search_long_term_memory(\n",
252-
" query=query,\n",
252+
" text=query,\n",
253253
" limit=limit\n",
254254
" )\n",
255255
" \n",

python-recipes/context-engineering/notebooks/section-4-optimizations/01_context_window_management.ipynb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -331,7 +331,7 @@
331331
" {\"role\": \"assistant\", \"content\": response.content}\n",
332332
" ])\n",
333333
" \n",
334-
" from agent_memory_client import WorkingMemory, MemoryMessage\n",
334+
" from agent_memory_client.models import WorkingMemory, MemoryMessage\n",
335335
" \n",
336336
" # Convert messages to MemoryMessage format\n",
337337
" memory_messages = [MemoryMessage(**msg) for msg in all_messages]\n",

python-recipes/context-engineering/notebooks/section-4-optimizations/03_grounding_with_memory.ipynb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@
151151
" \n",
152152
" # Search long-term memory for context\n",
153153
" memories = await memory_client.search_long_term_memory(\n",
154-
" query=user_message,\n",
154+
" text=user_message,\n",
155155
" limit=5\n",
156156
" )\n",
157157
" \n",
@@ -183,7 +183,7 @@
183183
" {\"role\": \"user\" if isinstance(m, HumanMessage) else \"assistant\", \"content\": m.content}\n",
184184
" for m in conversation_history\n",
185185
" ]\n",
186-
" from agent_memory_client import WorkingMemory, MemoryMessage\n",
186+
" from agent_memory_client.models import WorkingMemory, MemoryMessage\n",
187187
" \n",
188188
" # Convert messages to MemoryMessage format\n",
189189
" memory_messages = [MemoryMessage(**msg) for msg in messages_to_save]\n",
@@ -426,7 +426,7 @@
426426
"\n",
427427
"# Get all memories\n",
428428
"all_memories = await memory_client.search_long_term_memory(\n",
429-
" query=\"\",\n",
429+
" text=\"\",\n",
430430
" limit=20\n",
431431
")\n",
432432
"\n",

python-recipes/context-engineering/notebooks/section-4-optimizations/05_crafting_data_for_llms.ipynb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -438,7 +438,7 @@
438438
"\n",
439439
"# Get memories\n",
440440
"memories = await memory_client.search_long_term_memory(\n",
441-
" query=\"\", # Get all\n",
441+
" text=\"\", # Get all\n",
442442
" limit=20\n",
443443
")\n",
444444
"\n",
Lines changed: 63 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,63 @@
1+
#!/usr/bin/env python3
2+
"""
3+
Fix all query= to text= in search_long_term_memory calls across all notebooks.
4+
Also fix missing imports.
5+
"""
6+
7+
import json
8+
import glob
9+
from pathlib import Path
10+
11+
12+
def fix_notebook(notebook_path):
13+
"""Fix a single notebook."""
14+
with open(notebook_path, 'r') as f:
15+
nb = json.load(f)
16+
17+
modified = False
18+
for cell in nb['cells']:
19+
if cell['cell_type'] == 'code':
20+
new_source = []
21+
for line in cell['source']:
22+
original = line
23+
# Fix query= to text= in search_long_term_memory calls
24+
if 'search_long_term_memory' in line or (len(new_source) > 0 and 'search_long_term_memory' in ''.join(new_source[-3:])):
25+
line = line.replace('query=', 'text=')
26+
27+
# Fix missing imports
28+
if 'from agent_memory_client import WorkingMemory' in line:
29+
line = line.replace('from agent_memory_client import WorkingMemory', 'from agent_memory_client.models import WorkingMemory')
30+
if 'from agent_memory_client import MemoryMessage' in line:
31+
line = line.replace('from agent_memory_client import MemoryMessage', 'from agent_memory_client.models import MemoryMessage')
32+
33+
new_source.append(line)
34+
if line != original:
35+
modified = True
36+
cell['source'] = new_source
37+
38+
if modified:
39+
with open(notebook_path, 'w') as f:
40+
json.dump(nb, f, indent=2, ensure_ascii=False)
41+
f.write('\n')
42+
return True
43+
return False
44+
45+
46+
def main():
47+
notebooks_dir = Path(__file__).parent.parent / 'notebooks'
48+
49+
fixed_count = 0
50+
for notebook_path in notebooks_dir.glob('**/*.ipynb'):
51+
if '.ipynb_checkpoints' in str(notebook_path):
52+
continue
53+
54+
if fix_notebook(notebook_path):
55+
print(f"Fixed: {notebook_path.relative_to(notebooks_dir)}")
56+
fixed_count += 1
57+
58+
print(f"\nFixed {fixed_count} notebooks")
59+
60+
61+
if __name__ == '__main__':
62+
main()
63+

0 commit comments

Comments
 (0)