AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

remove print_total_cost variable and optimization of docstring format

Open lengweiping1983 opened this issue 2 years ago • 3 comments

Background

  1. The print_total_cost global variable is unnecessary, and the existing self.debug variable is directly used in the class.
  2. Wrong docstring total_budget parameter.
  3. optimization of docstring format.

Changes

  1. remove print_total_cost variable
  2. 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

lengweiping1983 avatar Apr 26 '23 04:04 lengweiping1983

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:

... and 16 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Apr 26 '23 05:04 codecov[bot]

I think this should be 3 different PRs.

hdkiller avatar Apr 26 '23 08:04 hdkiller

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

github-actions[bot] avatar Apr 26 '23 16:04 github-actions[bot]

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

vercel[bot] avatar Apr 26 '23 23:04 vercel[bot]

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

github-actions[bot] avatar Apr 26 '23 23:04 github-actions[bot]