AutoGPT
AutoGPT copied to clipboard
Added default actions functionality
Fixes #236 Related to #150
Background
This pull request introduces a new functionality to the Agent class, allowing a default number of authorizations to be set for the start_interaction_loop() method. During our work, we also addressed an issue with next_action_count, which was being treated as a string instead of an integer, causing a TypeError when attempting to decrement its value.
Changes
- Implemented a new
default_authorisation_countparameter in theAgentclass. - Updated the
__init__method of theAgentclass to castnext_action_countto an integer. - Ensured that
default_authorisation_countis also cast to an integer when updatingnext_action_count.
Documentation
The changes made in this pull request have been documented through in-code comments and this PR message.
Test Plan
- Run the application with various inputs to ensure the TypeError does not occur anymore.
- Verify that
next_action_countis correctly updated and decremented as intended during the interaction loop. - Test the new
default_authorisation_countfunctionality to ensure it works as intended.
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
@CatsMeow492 CI is red
@nponeccop CI green :)
CI is red
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
༼ つ ◕_ ◕ ༽つ GIVE MERGE ༼ つ ◕_ ◕ ༽つ
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
༼ つ ◕_ ◕ ༽つ ⊂(◕_ ◕ ⊂) ⊂༼ ◕_ ◕ ༽つ WHERE MERGE?
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
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
This is an interesting but multi-faceted PR, some of which would be better as a plugin and some of which has improperly resolved conflicts such as .env.template. We would be happy to have your contributions, but please submit unrelated changes separately. :)