AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

[Bugfix] Fix memory add

Open tusharmakkar08 opened this issue 1 year ago • 10 comments

Background

Adding memory add back

Changes

Memory add was removed from prompt during refactoring

PR Quality Checklist

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

tusharmakkar08 avatar Apr 19 '23 09:04 tusharmakkar08

Fixes https://github.com/Significant-Gravitas/Auto-GPT/issues/2509

tusharmakkar08 avatar Apr 19 '23 09:04 tusharmakkar08

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

github-actions[bot] avatar Apr 20 '23 00:04 github-actions[bot]

I believe this PR needs to be refactored according to the new structure introduced by the plugins merge.

hdkiller avatar Apr 20 '23 04:04 hdkiller

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

github-actions[bot] avatar Apr 20 '23 09:04 github-actions[bot]

@hdkiller Can you PTAL? I have resolved merge conflicts.

tusharmakkar08 avatar Apr 20 '23 09:04 tusharmakkar08

@richbeales Can you have a look?

tusharmakkar08 avatar Apr 20 '23 09:04 tusharmakkar08

Please run "black ." to fix formatting

richbeales avatar Apr 21 '23 05:04 richbeales

@richbeales Added black formatting. PTAL.

tmakkar-eightfold avatar Apr 21 '23 08:04 tmakkar-eightfold

Any updates on this? @richbeales

tmakkar-eightfold avatar Apr 26 '23 04:04 tmakkar-eightfold

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

github-actions[bot] avatar May 03 '23 23:05 github-actions[bot]

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

p-i- avatar May 05 '23 00:05 p-i-

This is incompatible with the new memory system, which is tracked in #3536. It may take a while to finish, but once it's done, should boost Auto-GPT's performance a huge deal.

Pwuts avatar Jun 14 '23 15:06 Pwuts