AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

Published 20 hours ago • verified publisher icon Significant-Gravitas

Chroma memory

Open jhouseke opened this issue 2 years ago • 12 comments

Background

Project needs more open source vector embedding db options.

Changes

Implemented Chroma database.

Documentation

Test Plan

PR Quality Checklist

  • [x] My pull request is atomic and focuses on a single change.
  • [X] I have thouroughly 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

jhouseke avatar Apr 11 '23 20:04 jhouseke

@jhouseke There are conflicts now

nponeccop avatar Apr 12 '23 18:04 nponeccop

Resolved conflicts, thank you for heads up

jhouseke avatar Apr 12 '23 19:04 jhouseke

It's pretty important to merge this. Reliance on a closed source service isn't a good outlook (though we're still dependent on OpenAI)

gptlang avatar Apr 15 '23 12:04 gptlang

@jhouseke There are conflicts now

nponeccop avatar Apr 15 '23 14:04 nponeccop

@jhouseke Any updates on this?

sadmuphin avatar Apr 15 '23 14:04 sadmuphin

resolving now

jhouseke avatar Apr 15 '23 18:04 jhouseke

PR no longer has conflicts. Anything else I need to be aware of?

jhouseke avatar Apr 15 '23 18:04 jhouseke

@jhouseke CI is red now.

nponeccop avatar Apr 17 '23 11:04 nponeccop

removed whitespace to resolve W293

EDIT: resolving other issues identified, standby

EDIT_2: all issues identified in most recent CI run resolved

jhouseke avatar Apr 17 '23 14:04 jhouseke

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

github-actions[bot] avatar Apr 17 '23 16:04 github-actions[bot]

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

github-actions[bot] avatar Apr 17 '23 18:04 github-actions[bot]

Any progress on this - would love to use chroma with autogpt?

HammadB avatar Apr 24 '23 20:04 HammadB

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

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 2, 2023 6:47pm

vercel[bot] avatar Apr 27 '23 13:04 vercel[bot]

Branch is up-to-date, awaiting merge

jhouseke avatar Apr 27 '23 14:04 jhouseke

@nponeccop any chance this can get merged in? thanks!

jhouseke avatar May 02 '23 18:05 jhouseke

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-

Hey, I've marked this as don't merge until the Memory Fixes are in. Sorry to keep it on hold longer, just no real way to test/validate functionality until the fixes are in

ntindle avatar May 25 '23 15:05 ntindle

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

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

There is currently no prospect of this getting merged, so unfortunately I'll have to close it. The structure of the codebase and the application has changed a lot, so merging it would be a lot of work, and before we start adding vector stores, we need a good high-level design for memory use. This will be tracked in #3536, which needs updating as I write this.

There's a good chance Chroma will eventually be added, and we really do appreciate the effort that went into this PR. If you're still interested in (contributing to) the project, keep an eye on the #dev-autogpt channel. We've been making good progress in the past weeks, and my current #1 todo is making sure contributors can meaningfully contribute again.

Pwuts avatar Sep 07 '23 16:09 Pwuts