babyagi icon indicating copy to clipboard operation
babyagi copied to clipboard

Task Manager / Proper Classes

Open peteoxenham opened this issue 1 year ago • 3 comments

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.

peteoxenham avatar Apr 06 '23 01:04 peteoxenham

I think this is the best approach. Thanks for doin this

Gogolian avatar Apr 06 '23 08:04 Gogolian

I don't have GPT4 API access, it should work, but somebody please test and report back. Thank you!

peteoxenham avatar Apr 06 '23 16:04 peteoxenham

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

Gogolian avatar Apr 06 '23 20:04 Gogolian