-
Notifications
You must be signed in to change notification settings - Fork 57
Feat/context eng agent v2 #111
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.
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
Draft
nkanu17
wants to merge
138
commits into
main
Choose a base branch
from
feat/context-eng-agent-v2
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
138 commits
Select commit
Hold shift + click to select a range
0d58062
Add context engineering course with Redis University Class Agent
abrookins 2064e61
Temporarily ignore context engineering notebooks in CI
abrookins 6be84e5
Fix notebooks to work in CI environments
abrookins 73c9161
Fix final import issue in 03_project_overview notebook
abrookins 065e91a
Fix real issues: Install package in CI and use real classes
abrookins 2f014d5
Fix RedisVL API compatibility issue in memory retrieval
abrookins 3011f52
Fix RedisVL API format change - handle both dict and object results
abrookins 7b5059f
Clean up notebook 01: Remove installation guards and update memory te…
abrookins 8f53551
Implement working memory with long-term extraction strategy awareness
abrookins 4c80a28
Complete Context Engineering course with all 15 notebooks and referen…
abrookins 8cb9c10
Temporarily ignore context engineering notebooks in CI
abrookins 8722b48
Revert: Remove context engineering notebooks from ignore list
abrookins e7ce2ba
Fix notebook imports: MemoryManager -> MemoryClient
abrookins db78b54
Add agent-memory-client to dependencies
abrookins 8abb21d
Fix working memory notebook to use actual MemoryClient API
abrookins 3f7ed02
Fix notebook cell dependencies for independent execution
abrookins 65ae681
Fix MemoryAPIClient initialization to use MemoryClientConfig
abrookins 6804da9
Fix method name: set_working_memory -> put_working_memory
abrookins a07e72c
Comment out memory_manager calls in notebooks
abrookins b68536e
Convert memory_manager cells to markdown in 02_role_of_context_engine
abrookins 64ee02c
Fix memory_type -> memory_types parameter in notebooks
abrookins cff357a
Fix count_tokens function dependencies in context window notebook
abrookins 19e671f
Fix put_working_memory() call signature
abrookins 0e71885
Fix MemoryRecord creation to include required id field
abrookins 7efbe57
Add Docker Compose setup for Agent Memory Server
abrookins 20ec0d1
Fix Agent Memory Server Docker image path
abrookins e3111a8
Fix MemoryRecord creation in save_working_memory
abrookins 8ff0982
Add get_or_create_working_memory and update notebooks
abrookins 80a71f6
Improve OpenAI API key handling and CI debugging
abrookins 0fbbc06
Make Agent Memory Server startup more lenient in CI
abrookins 5cd45eb
Fix LOG_LEVEL environment variable for Agent Memory Server
abrookins b1a051c
Fix get_or_create_working_memory to unpack tuple return value
abrookins 71f93c0
Fix create_memory to handle tuple return from create_long_term_memories
abrookins ad8de72
Remove memory_client wrapper and use MemoryAPIClient directly
abrookins 0482676
Fix agent.py to use actual MemoryAPIClient API
abrookins 93a3559
Fix tools.py to use actual MemoryAPIClient API
abrookins 4e1c4c2
Update notebooks to use MemoryAPIClient directly (partial)
abrookins 39290e6
Add comprehensive memory client migration documentation
abrookins ff2c4ac
Fix all save_working_memory calls to use put_working_memory
abrookins bd66460
Fix remaining import issues in notebooks
abrookins 07ac032
Fix section-3-memory/02_long_term_memory.ipynb API calls
abrookins 0f19481
Fix remaining API issues in section-3-memory notebooks
abrookins 30ea09e
Fix ClientMemoryRecord import - use agent_memory_client.models
abrookins 01b693a
Fix remaining query= to text= in 02_long_term_memory.ipynb
abrookins 7780258
Fix memory_type parameter to use MemoryType filter object
abrookins d6b36b0
Fix ClientMemoryRecord memory_type - use string not MemoryType filter
abrookins 6f67685
Fix ALL remaining query= to text= in notebooks
abrookins 61a6c3a
Fix missing MemoryClientConfig import in 01_working_memory notebook
abrookins da51403
Add missing user_id parameter to get_or_create_working_memory calls
abrookins 2675d88
Fix missing comma in get_or_create_working_memory call
abrookins 85cbe9e
Fix user_id consistency in 01_working_memory notebook
abrookins 015c4e3
Fix syntax errors and API usage in section-3 notebooks
abrookins e5f5b79
Fix tool invocation in 04_memory_tools notebook
abrookins 55e19d2
Fix list comprehension in 03_memory_integration notebook
abrookins 8da9cc7
Add missing user_id to all get_or_create_working_memory calls in 03_m…
abrookins 288f858
Fix all API usage issues in 03_memory_integration notebook
abrookins 5c77fd9
Fix get_or_create_working_memory issue in 03_memory_integration
abrookins bc37b82
Upgrade agent-memory-client to 0.12.3
abrookins 1677e59
Fix redisvl API compatibility in course_manager
abrookins 1ee634b
Add get_all_courses method to CourseManager
abrookins 45353bf
Add missing MemoryClientConfig import to section-4 notebooks
abrookins 46bf6bc
Fix remaining issues in section-4 notebooks
abrookins 1168b32
Fix final issues in section-4 notebooks
abrookins a92fe1e
Fix Redis get() calls in 05_crafting_data_for_llms
abrookins 2092f1c
Fix KeyError in 05_crafting_data_for_llms
abrookins 0a81a94
Fix len(memories) in 05_crafting_data_for_llms
abrookins 9aae1c1
Fix memories slicing in 05_crafting_data_for_llms
abrookins f91263b
Update Redis version to 8.2 in GitHub Actions workflows
abrookins e113888
Remove OpenAI API key check and logging from workflow
abrookins 3cc032a
Use redis:8.2 image in test.yml workflow
abrookins b3bec17
Add search_courses_tool to demonstrate catalog view + RAG pattern
abrookins a0680f1
Change to get_course_details tool that retrieves by course code
abrookins ffc8389
Add detailed explanation of data integration challenges
abrookins fe4149f
Fix JSON syntax error in 05_crafting_data_for_llms.ipynb
abrookins a0cf9a0
Clarify LLM control statement in automatic extraction section
abrookins 53fb3ef
Clarify that it's the application's LLM that has control in tool-base…
abrookins 0de2ddb
Add performance tradeoffs to memory extraction comparison
abrookins 7637eff
Update 04_memory_tools to use built-in memory client tool schemas
abrookins c4c7501
Keep LangChain/LangGraph pattern while using memory client tools
abrookins 036ff05
Use memory client's built-in LangChain/LangGraph integration
abrookins af26356
Update reference agent to use memory client's LangChain integration
abrookins 6bc85d4
Add schema printing to memory tools notebook for debugging
abrookins 5396372
Add schema printing to memory tools notebook
abrookins 22ceafa
Catch and return tool validation errors to LLM
abrookins 8c10741
Fix create_memory_tools call in working memory notebook
abrookins bab8f38
Simplify setup and clean up notebooks
abrookins 07c1e91
Bump agent-memory-client to 0.12.6
abrookins a9b8a7a
Reorganize notebook setup and update agent-memory-server to 0.12.3
abrookins 96a1e2a
Fix notebook failures: remove non-existent memory_index_name and meta…
abrookins bf0c001
Notebook cleanups: replace print-based informational cells with Markd…
abrookins 1cec8b3
Fix malformed JSON in 02_project_overview.ipynb to unblock nbval coll…
abrookins a0d7f33
context-engineering: use Redis 8, fix 8088 refs, add tests
abrookins ba1770b
Add revised tool definition notebook for agent fundamentals
nkanu17 aaa427f
Add initial notebooks for fundamentals and RAG foundations
nkanu17 0fd6cf2
Add Section 1 fundamentals and Section 2 RAG notebooks
nkanu17 c59b22b
Add Section 3 memory architecture notebooks (grounding and memory int…
nkanu17 56d0d89
Revamp Section 1 introduction with enhanced context engineering content
nkanu17 6a32632
Add comprehensive RAG retrieved context notebook
nkanu17 db9a091
Remove obsolete notebooks from enhanced-integration and old sections
nkanu17 30837a7
Add Section 3 memory fundamentals and memory-enhanced RAG notebooks
nkanu17 b1f095f
Enhance memory fundamentals notebook and add Agent Memory Server setu…
nkanu17 340136f
Refactor memory fundamentals notebook for improved long-term memory c…
nkanu17 f40a873
Complete Section 3 memory architecture with enhanced RAG integration …
nkanu17 6cbc71b
Remove duplicate introduction and memory notebooks
nkanu17 ab6e32c
Revamp Section 1 with context types deep dive and LangGraph fundamentals
nkanu17 bbfb8a5
Add Section 4 Redis University course advisor agent notebook
nkanu17 5114a7c
Enhance system instructions, RAG, and memory notebooks with productio…
nkanu17 453a3b0
Remove obsolete revised_notebooks directory
nkanu17 dc80135
Complete Section 5 with performance, scaling, and production readines…
nkanu17 27b0568
Add comprehensive memory management and compression strategies notebook
nkanu17 8ae5357
Polish all notebooks with improved formatting and production quality …
nkanu17 bfc71f4
Updates to previous notebooks to keep in commit history
nkanu17 f3bf5e4
* Update readme, course_summary based on changes.
nkanu17 8dfd7ad
Rename notebooks_v2 to notebooks and update documentation
nkanu17 ea4efc2
Remove nk_scripts dir from git
nkanu17 dcf7ba7
Clean up notebooks directory - remove development/testing files
nkanu17 4b95635
Remove test scripts and development files from root directory
nkanu17 7a32607
Restore original 02_full_featured_agent.ipynb and remove Copy1 version
nkanu17 b4c0b1a
Remove development documentation files
nkanu17 ee464e4
Remove test documentation and scripts
nkanu17 204a63a
Remove development artifacts from reference-agent
nkanu17 f2ce19f
Remove unused .env.example.revised and archive directories
nkanu17 b676ea5
Remove development artifacts from notebooks directory
nkanu17 d88be0d
Rename Section 1: Context Engineering Foundations with updated notebo…
nkanu17 a5ddcb2
Rename Section 2: Retrieved Context Engineering with updated notebook…
nkanu17 266db7e
Rename Section 3: Memory Systems for Context Engineering with updated…
nkanu17 4bc5fd4
Remove development directories from git tracking (keep files locally)
nkanu17 cee408c
Remove personal development files from git tracking (keep locally)
nkanu17 71884c2
Rename Section 4: Integrating Tools and Agents with updated notebook …
nkanu17 578dff5
Format notebooks
nkanu17 8647c22
Add advanced context engineering with chunking notebook
nkanu17 d0ffbcb
Merge Section 2 & 2.5: Unified Retrieved Context Engineering
nkanu17 484e9da
Remove merge analysis docs from git tracking
nkanu17 fd8c382
Remove section 5 notebooks
nkanu17 62bf716
Removed vector alogrithm benchmark notebook from git
nkanu17 0271fad
De-emphasize production language, resolve TODOs, and clean up files
nkanu17 96bf0f5
Reword some content
nkanu17 2505ab9
Add architecture diagram
nkanu17 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| # OpenAI API Key (required to pass to the API container) | ||
| OPENAI_API_KEY=your-openai-api-key-here |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| venv | ||
| .env |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider pinning to a specific patch version (e.g.,
redis:8.2.2) for reproducible CI builds. Usingredis:8.2means the tag could point to different patch versions over time, potentially causing inconsistent test results.