Skip to content

feat: reorganizer config code and add remove dup nodes for playground-demo #135

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 15 commits into from
Jul 22, 2025
Merged

feat: reorganizer config code and add remove dup nodes for playground-demo #135

merged 15 commits into from
Jul 22, 2025
+308 −142

Conversation

fridayL
Copy link
Collaborator

@fridayL fridayL commented Jul 22, 2025 •
edited
Loading

Description

Summary: reorganizer config code and remove dup nodes when get all tree memory
Reviewer: @J1awei-Yang

Checklist:

  • I have performed a self-review of my own code | 我已自行检查了自己的代码
  • I have commented my code in hard-to-understand areas | 我已在难以理解的地方对代码进行了注释
  • I have added tests that prove my fix is effective or that my feature works | 我已添加测试以证明我的修复有效或功能正常
  • I have created related documentation issue/PR in MemOS-Docs (if applicable) | 我已在 MemOS-Docs 中创建了相关的文档 issue/PR（如果适用）
  • I have linked the issue to this PR (if applicable) | 我已将 issue 链接到此 PR（如果适用）
  • I have mentioned the person who will review this PR | 我已提及将审查此 PR 的人

@fridayL fridayL requested a review from J1awei-Yang July 22, 2025 06:08
J1awei-Yang
J1awei-Yang reviewed Jul 22, 2025
View reviewed changes
Copy link
Collaborator

@J1awei-Yang J1awei-Yang left a comment

Choose a reason for hiding this comment

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

Since it's not one of the example scripts, would it be better to use logger for output instead?

@fridayL
Copy link
Collaborator Author

fridayL commented Jul 22, 2025

Since it's not one of the example scripts, would it be better to use logger for output instead?

yes, i have change logger

@fridayL fridayL requested a review from J1awei-Yang July 22, 2025 09:29
@fridayL fridayL merged commit e2d34d8 into MemTensor:dev Jul 22, 2025
20 checks passed
tangg555 pushed a commit to tangg555/MemOS that referenced this pull request Jul 29, 2025
@fridayL @tangg555
feat: reorganizer config code and add remove dup nodes for playground…
660f956 
…-demo (MemTensor#135)

* feat: update config

* fix:dim

* change dim

* fix:change default db

* fix：delay

* fix:len

* fix:change recently mem size

* fix:dup node error

* fix: remove mock_data

* fix: change config

* feat: reorganize code

* add: add json parse for en

* fix:change user_id

* fix: logger info
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@J1awei-Yang J1awei-Yang Awaiting requested review from J1awei-Yang

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fridayL @J1awei-Yang