-
Notifications
You must be signed in to change notification settings - Fork 43.3k
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
Refactor/move functions in app to agent #4957
Refactor/move functions in app to agent #4957
Conversation
✅ Deploy Preview for auto-gpt-docs canceled.
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #4957 +/- ##
==========================================
- Coverage 51.01% 51.00% -0.01%
==========================================
Files 119 118 -1
Lines 4895 4884 -11
Branches 643 643
==========================================
- Hits 2497 2491 -6
+ Misses 2217 2212 -5
Partials 181 181
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, aside from the merge conflict it's going to give me on #4799 :P
* Add links to github issues in the README and clarify run instructions * Move things only used by the agent out of app.py and into the agent module * Fix busted dynamic import
Background
We have some functions in autogpt.app that are literally only called by the Agent. I want the autogpt.app namespace for like actual application code.
Changes
test_agent.py
and use the agent fixture defined in the main conftest.pyDocumentation
X
Test Plan
CI
PR Quality Checklist