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

Deep lake vector store #2677

Closed

Conversation

adolkhan
Copy link

@adolkhan adolkhan commented Apr 20, 2023

Background

DeepLake can be used as a new optional memory backend. It's a open-source software that can be easily installed with pip install deeplake

Changes

  • existing function is not modified.
  • add DeepLake as a new optional memory backend.
  • part of document about memory backend is updated.

Documentation

New module and code is full documented.
A use guide about new feature is add to README.md.

Test Plan

  • Auto-GPT works properly in DeepLake memory backend.
  • Auto-GPT using local memory works properly too.

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

@k-boikov k-boikov added enhancement New feature or request Needs Benchmark This change is hard to test and requires a benchmark needs discussion To be discussed among maintainers potential plugin This may fit better into our plugin system. function: memory labels Apr 20, 2023
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Apr 22, 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.

@mikayelh
Copy link

mikayelh commented May 4, 2023

hey @k-boikov, would you mind taking a look at this? Deep Lake is open source, AutoGPT would benefit from more open source vector stores/memory. :)

@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

@Pwuts
Copy link
Member

Pwuts commented Jun 14, 2023

This has been rendered incompatible by the ovehaul of the memory storage system (#4208). Further work on the memory system is tracked in #3536; once that's completed we'll start accepting contributions to support more memory stores again. For now, having only one backend to support allows us to develop and iterate more easily.

Thanks for submitting! I'll try to remember this one for when we're done with memory work.

@Pwuts Pwuts closed this Jun 14, 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 enhancement New feature or request function: memory Needs Benchmark This change is hard to test and requires a benchmark needs discussion To be discussed among maintainers potential plugin This may fit better into our plugin system. size/l
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants