-
Notifications
You must be signed in to change notification settings - Fork 44.6k
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
Remove dead agent manager #4900
Remove dead agent manager #4900
Conversation
✅ Deploy Preview for auto-gpt-docs canceled.
|
Sub-agents are still listed as an available resource in prompt_settings.yml |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #4900 +/- ##
=======================================
Coverage 50.55% 50.56%
=======================================
Files 116 115 -1
Lines 4860 4790 -70
Branches 657 636 -21
=======================================
- Hits 2457 2422 -35
+ Misses 2219 2192 -27
+ Partials 184 176 -8
☔ View full report in Codecov by Sentry. |
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. |
You changed AutoGPT's behaviour. The cassettes have been updated and will be merged to the submodule when this Pull Request gets merged. |
You changed AutoGPT's behaviour. The cassettes have been updated and will be merged to the submodule when this Pull Request gets merged. |
You changed AutoGPT's behaviour. The cassettes have been updated and will be merged to the submodule when this Pull Request gets merged. |
* Remove dead agent manager --------- Co-authored-by: Reinier van der Leer <[email protected]>
* Remove dead agent manager --------- Co-authored-by: Reinier van der Leer <[email protected]>
Background
The AgentManager is dead code (and a confusing abstraction to begin with). It is also one of the last few singletons in the system.
Changes
Documentation
N/A
Test Plan
CI
PR Quality Checklist