AutoGPT
AutoGPT copied to clipboard
remove print_total_cost variable and optimization of docstring format
Background
- The print_total_cost global variable is unnecessary, and the existing self.debug variable is directly used in the class.
- Wrong docstring total_budget parameter.
- optimization of docstring format.
Changes
- remove print_total_cost variable
- optimization of docstring format.
Documentation
Test Plan
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
Codecov Report
Patch coverage has no change and project coverage change: -7.80 :warning:
Comparison is base (
7a006af) 51.00% compared to head (3797eb6) 43.21%.
Additional details and impacted files
@@ Coverage Diff @@
## master #3291 +/- ##
==========================================
- Coverage 51.00% 43.21% -7.80%
==========================================
Files 65 65
Lines 2976 2976
Branches 504 504
==========================================
- Hits 1518 1286 -232
- Misses 1334 1615 +281
+ Partials 124 75 -49
| Impacted Files | Coverage Δ | |
|---|---|---|
| autogpt/api_manager.py | 93.18% <ø> (-2.28%) |
:arrow_down: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
I think this should be 3 different PRs.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| docs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Apr 26, 2023 11:45pm |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.