Skip to content
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

feat: mem0 ai memory block #9285

Open
wants to merge 38 commits into
base: dev
Choose a base branch
from
Open

feat: mem0 ai memory block #9285

wants to merge 38 commits into from
+702 −4

Conversation

ntindle
Copy link
Member

@ntindle ntindle commented Jan 16, 2025 •
edited
Loading

We want to have some more intelligent and less user managed memory methods, so we add mem0

Changes 🏗️

  • Adds user_id to kwargs for blocks
  • Add mem0 blocks

Checklist 📋

  • document adding user_id to kwargs for blocks
  • Add run and agent Id as optional checkboxes that will be passed down to mem0

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • Build and submit an agent to @Torantulino and the marketplace for a personal AI tutor based on recommendations from the mem0 team

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end platform/blocks labels Jan 16, 2025
@netlify Netlify
Copy link

netlify bot commented Jan 16, 2025 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit c785eac
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/67974bfcc48a7f000895d94b

@netlify Netlify
Copy link

netlify bot commented Jan 16, 2025 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit c785eac
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/67974bfcb33cb20008e63909

@github-actions github-actions bot added size/xl and removed size/l labels Jan 16, 2025
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jan 16, 2025
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jan 18, 2025
@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jan 21, 2025
@github-actions GitHub Actions
Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

@ntindle ntindle enabled auto-merge January 23, 2025 03:19
@ntindle ntindle requested a review from majdyz January 24, 2025 13:16
majdyz
majdyz reviewed Jan 24, 2025
View reviewed changes
autogpt_platform/backend/backend/executor/manager.py Outdated Show resolved Hide resolved
autogpt_platform/backend/backend/blocks/basic.py Show resolved Hide resolved
autogpt_platform/backend/backend/blocks/basic.py Outdated Show resolved Hide resolved
autogpt_platform/backend/backend/blocks/mem0.py
credentials: CredentialsMetaInput[
Literal[ProviderName.MEM0], Literal["api_key"]
] = CredentialsField(description="Mem0 API key credentials")
content: Union[str, list[dict[str, str]]] = SchemaField(
Copy link
Contributor

Choose a reason for hiding this comment

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

How is this going to be rendered? in what format we should provide for this value

Copy link
Member Author

@ntindle ntindle Jan 26, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

image
it renders as this, but it should be a string or an ai block llm output. I think string is okay, but not sure how I'd improve this

autogpt_platform/backend/backend/blocks/mem0.py Outdated Show resolved Hide resolved
autogpt_platform/backend/backend/blocks/mem0.py Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jan 24, 2025
@github-actions GitHub Actions
Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jan 24, 2025
@ntindle ntindle requested a review from majdyz January 26, 2025 12:59
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jan 26, 2025
@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jan 27, 2025
@github-actions GitHub Actions
Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dev-Khant Dev-Khant Dev-Khant left review comments

@Bentlybro Bentlybro Awaiting requested review from Bentlybro Bentlybro is a code owner automatically assigned from Significant-Gravitas/maintainers

@aarushik93 aarushik93 Awaiting requested review from aarushik93 aarushik93 is a code owner automatically assigned from Significant-Gravitas/maintainers

@majdyz majdyz Awaiting requested review from majdyz

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation platform/backend AutoGPT Platform - Back end platform/blocks platform/frontend AutoGPT Platform - Front end Review effort [1-5]: 4 size/xl
Projects
Frontend
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ntindle @Dev-Khant @majdyz