Delete duplicate method declaration

[Androbin]

Background

Method declaration "print_assistant_thoughts" is obscured by a declaration of the same name
Method declaration "test_count_message_tokens_invalid_model" is obscured by a declaration of the same name
Method declaration "test_invalid_json_leading_sentence_with_gpt" is obscured by a declaration of the same name
Pylance(reportGeneralTypeIssues)

Changes

Delete obscured method declarations

Documentation

Test Plan

PR Quality Checklist

• My pull request is atomic and focuses on a single change.
• I have thoroughly tested my changes with multiple different prompts.
• I have considered potential risks and mitigations for my changes.
• I have documented my changes clearly and comprehensively.
• I have not snuck in any "extra" small tweaks changes

[jakubbober]

Hi @Androbin, have you checked that in the test indeed the NotImplementedError is thrown, rather than the KeyError? It's the only difference between the two copies of the function.

[Androbin]

@jakubbober The second declaration obscures the first, and all tests pass, therefore the second test passes, and since only one (type of) Exception can be thrown at the same time, the first declaration can be safely deleted.

[github-actions]

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

[github-actions]

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

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -7.98 ⚠️

Comparison is base (8670b30) 50.83% compared to head (6135526) 42.86%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2033      +/-   ##
==========================================
- Coverage   50.83%   42.86%   -7.98%     
==========================================
  Files          65       65              
  Lines        2935     2935              
  Branches      493      493              
==========================================
- Hits         1492     1258     -234     
- Misses       1319     1615     +296     
+ Partials      124       62      -62     

see 15 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

[github-actions]

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

[p-i-]

This is a mass message from the AutoGPT core team.
Our apologies for the ongoing delay in processing PRs.
This is because we are re-architecting the AutoGPT core!

For more details (and for infor on joining our Discord), please refer to:
https://github.com/Significant-Gravitas/Auto-GPT/wiki/Architecting

[Pwuts]

This has been fixed by now. Thanks for submitting!