babyagi
babyagi copied to clipboard
Task Manager / Proper Classes
First off - I realize this is a bit of a doozy of a PR - I don't necessarily intend for it to get merged as-is because I realize it will break a lot of existing PRs, but I'd love to have a discussion about it.
I moved everything into classes. The intent here was to keep the exact same functionality as the existing main branch, but make it easier to work and collaborate on.
I think since the point of this project, as I understand it, is to encourage collaboration on a framework across multiple different thoughts and experiments, this is really the way to go to make future innovation easier.
Pros:
- Things are in classes, so core functionality can be built out and rearranged easier
- Creating an async version (my next plan) is far easier with this model
- This sets the basis for re-scoping contexts for additional threads (<-- this is the part I really care about)
Cons:
- I didn't comment it well, would love some help/feedback with that
- We're nuking a lot of code
Concerns:
- I'm not sure if the folder structure is logical/properly categorized
- I would like a few others to look it over to make sure I didn't break anything. FWIW, it seems equivalent in all my tests.
I think this is the best approach. Thanks for doin this
I don't have GPT4 API access, it should work, but somebody please test and report back. Thank you!
You should be able to test it without GPT4 access, if you have free account you should be able to access model text-davinci-003 which is still gpt-3.5