AutoGPT
AutoGPT copied to clipboard
augment and make LocalCache memory/embeddings pluggable
Background
The LocalCache memory backend implemented in https://github.com/Torantulino/Auto-GPT/pull/372 used a handrolled implementation of a storage engine, and the embedding strategy was pinned to text-embedding-ada-002
which we may want to change/does cost money/latency
Changes
- reimplemented LocalCache using Chroma - which also runs either in-memory or persisted, but is well tested and managed.
- made embeddings swappable to ANY openai embedding model - and uses the Chroma default SentenceTransformers which costs no money
Documentation
Test Plan
testing it locally
no new tests needed since this is a strict upgrade of LocalCache.
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
its ready for review :)
great addition @sw-yx ! I hope it gets merged soon.
@sw-yx There are conflicts
whaaat i just made this PR yesterday and there are conflicts haha. will fix!
We move fast, new conflicts can happen every 5 minutes here :)
@nponeccop i think i fixed the conflict!
re-fixed new conflicts! @nponeccop what typically happens next?
@sw-yx There are conflicts now
@nponeccop as far as i can tell i fixed the conflicts as of commit a6928c2. i've just updated my fork to current master but still not seeing any conflicts as of right now!
@sw-yx There are both conflicts and CI failure now
@nponeccop fixed! mind running CI again?
@sw-yx CI fails
@nponeccop ok i have addressed the flake issues. i'd be happy to work on better doucmenting contributor guide next so that others dont have this issue
@sw-yx There are conflicts now
wew that was a big merge. i've updated again.
@sw-yx There are (minor) conflicts again
re-updated (and checked flake tests)
@nponeccop re-updated
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
thank you bot very cool
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
resolved
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
updated my fork but not showing on this PR yet for some reason
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
re updated