-
Notifications
You must be signed in to change notification settings - Fork 0
Long-Term Memory per User #239
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
Open
kkwan0
wants to merge
46
commits into
main
Choose a base branch
from
172-long-term-memory-per-user
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.
Open
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
d9d412d
delete ruff_cache. Added to the gitignore
Ar-temis 7337c3d
Adding mem0 to the project toml, added the mem0 collection name to co…
Ar-temis 4048235
Adding the long term user memory tool
Ar-temis 1249b71
Deleted unused dspy modules
Ar-temis e91e03e
renamed conversation_memory.py to memory.py. added permanent memory to
Ar-temis 73c6552
Solving circulat import problem in planner
Ar-temis 53c4a04
idk what's going on
Ar-temis 62c879b
Refactor database configuration variable names for consistency, confi…
kkwan0 92e4e10
Added new Phoenix project for #172
kkwan0 3ceaaa7
Added debug for memory_tool, but main issue is that it's timing out w…
kkwan0 f63249d
Revert "idk what's going on"
kkwan0 79c7f20
I accidentally reverted a commit instead of reverting to it
kkwan0 ad4bc49
Fixed store_memory error on my side
kkwan0 07ddfcb
Trying to update the mem0 prompt. by adding the specific tools that i…
kkwan0 64b0bde
Conversation is properly recorded instead of gen.
Ar-temis a59da31
Permanent memory agent properly receives all the stored memories.
Ar-temis e0b02e3
Fixing opentelemetry recording in memory.py
Ar-temis f8eda3a
returns memory_id now
kkwan0 a404c6c
Merge remote-tracking branch 'refs/remotes/origin/172-long-term-memor…
kkwan0 eaa8efc
merged commits?
kkwan0 ed8e0cb
removed debug lines
kkwan0 c5db21f
used idx -> memory_id mapping
kkwan0 0a867f9
Agent can now store metadata(I need to refine what it stores). Also f…
kkwan0 85b8b94
Stores metadata in the memory, as well as returns it when searching
kkwan0 3e67b4a
Basic metadata filtering implemented. Works on searches
kkwan0 2a2e2ff
Add custom fact extraction prompt and added start to cleaning memory …
kkwan0 6b89a1f
When searching memories it includes access count as well as time last…
kkwan0 8ffb48a
I need to test but I believe that there is periodic memory cleanup
kkwan0 36e4bd3
Verified that fact extraction is working, as well as implemented a la…
kkwan0 0a40b79
Refined memory cleanup by seperating memories into short and long ter…
kkwan0 140334b
Merge branch 'main' into 172-long-term-memory-per-user
Ar-temis 28525c3
Reverted phoenix project name
kkwan0 292597a
Black reformatting for linter
kkwan0 567ee3c
Revert "Black reformatting for linter"
kkwan0 839db49
Only formatted those two files
kkwan0 5ca48dc
Fixed linter format issues
kkwan0 cc0400d
final black format i hope
kkwan0 b229a89
made docstring shorter
kkwan0 5d3960e
Merge branch 'main' into 172-long-term-memory-per-user
kkwan0 ba4c5b6
Should pass lint. I merged cuz it had a merge conflict with main
kkwan0 4cd786c
I hate this thing
kkwan0 87e45d6
FastAPI endpoints, need to work out some structural things that might…
kkwan0 e829e1f
Potential fix for pull request finding 'Unused import'
Ar-temis 689d355
coderabbit proposed changes(modified prompt, datetime, as well as oth…
kkwan0 60c5c38
Merge branch '172-long-term-memory-per-user' of github.com:Edge-Intel…
kkwan0 5ebaae5
Revert "FastAPI endpoints, need to work out some structural things th…
kkwan0 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
Some comments aren't visible on the classic Files Changed page.
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,6 @@ | ||
|
|
||
| .harper-dictionary.txt | ||
| .ruff_cache | ||
|
|
||
| .idea/ | ||
| .vscode/ | ||
|
|
||
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
131 changes: 0 additions & 131 deletions
131
chatdku/chatdku/core/dspy_classes/conversation_memory.py
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.