Skip to content

fix: fixed list of type in EXTRACTION_PROMPT_PART_1 #178

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
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

CostantinoEsposito89
Copy link

Description

Corrected the EXTRACTION_PROMPT_PART_1 prompt, the list of TextualMemoryMetadata types only accepts: iteral[‘procedure’, ‘fact’, ‘event’, “opinion”, “topic”, ‘reasoning’] | None.
The LLM must not generate anything else.
Summary: (summary)

Reviewer: @fridayL

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 CaralHsi July 28, 2025 12:22
@CaralHsi CaralHsi changed the base branch from main to dev August 1, 2025 06:49
Copy link
Collaborator

@CaralHsi CaralHsi left a comment

Choose a reason for hiding this comment

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

Look good

@CaralHsi
Copy link
Collaborator

CaralHsi commented Aug 3, 2025

Hi, please resolve the conflicts. Thanks for your contribution:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants