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.

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

Added the Ability to change Memory Backend Types. #3737

Closed
wants to merge 1 commit into from
Closed

Added the Ability to change Memory Backend Types. #3737

wants to merge 1 commit into from

Conversation

frankymcgee
Copy link

@frankymcgee frankymcgee commented May 3, 2023

• Updated .env.template file
• Added new .bat and .sh files for using DEPENDS_ON Variable in .env file • created new docker-compose files for redis and weaviate • modified original docker-compose.yml files redis settings • Added ability

Background

Changes

Documentation

Test Plan

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

• Updated .env.template file
• Added new .bat and .sh files for using DEPENDS_ON Variable in .env file
• created new docker-compose files for redis and weaviate
• modified original docker-compose.yml files redis settings
• Added ability
@vercel
Copy link

vercel bot commented May 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) May 3, 2023 1:07pm

@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@p-i-
Copy link
Contributor

p-i- commented May 5, 2023

This is a mass message from the AutoGPT core team.
Our apologies for the ongoing delay in processing PRs.
This is because we are re-architecting the AutoGPT core!

For more details (and for infor on joining our Discord), please refer to:
https://github.com/Significant-Gravitas/Auto-GPT/wiki/Architecting

@k-boikov k-boikov added needs discussion To be discussed among maintainers docker function: memory labels May 14, 2023
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label May 16, 2023
@github-actions
Copy link
Contributor

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

@Pwuts
Copy link
Member

Pwuts commented Jun 7, 2023

Thanks for submitting. Since all the old memory backends were removed as a part of the ongoing re-arch (see #3536), it no longer makes sense to merge this. I'll make a note to incorporate this kind of easy install when we get to adding back some of these memory providers.

@Pwuts Pwuts closed this Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts Automatically applied to PRs with merge conflicts docker function: memory needs discussion To be discussed among maintainers size/xl
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants