AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

Title: Add model_name attribute to Agent class and update docstrings

Open alexanderulanch opened this issue 2 years ago • 6 comments

Description:

  • Added model_name attribute to the Agent class to specify the AI model used for generating responses.
  • Included model_name as a parameter in the Agent's init method and passed it to the chat_with_ai function.
  • Added a docstring to the start_interaction_loop method to describe its functionality.
  • Updated the Agent class docstring to include the model_name attribute.

Background

Changes

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

alexanderulanch avatar Apr 22 '23 22:04 alexanderulanch

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

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

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

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

Codecov Report

Patch coverage: 9.09% and project coverage change: -7.46 :warning:

Comparison is base (2b5852f) 45.09% compared to head (4c9d2b5) 37.64%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2955      +/-   ##
==========================================
- Coverage   45.09%   37.64%   -7.46%     
==========================================
  Files          60       60              
  Lines        2856     2856              
  Branches      472      472              
==========================================
- Hits         1288     1075     -213     
- Misses       1463     1719     +256     
+ Partials      105       62      -43     
Impacted Files Coverage Δ
autogpt/agent/agent.py 12.74% <0.00%> (-28.44%) :arrow_down:
autogpt/chat.py 24.28% <20.00%> (-45.72%) :arrow_down:

... and 14 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.

: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 23 '23 03:04 codecov[bot]

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

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

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

p-i- avatar May 05 '23 00:05 p-i-

The architecture for agents is currently being reworked see: https://github.com/Significant-Gravitas/Auto-GPT/discussions/3856#discussioncomment-5818746

anonhostpi avatar May 05 '23 19:05 anonhostpi

Most of this PR has been superseded or replaced by other PRs already. It looks good though, thanks for submitting! We won't be merging this because it is so far out of date, but we invite you to join our Discord server to discuss and coordinate further if you are still interested in contributing to the project!

Pwuts avatar Jul 07 '23 06:07 Pwuts