Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CLI args for ai_name, ai_role, and ai_goals #3250

Merged
merged 20 commits into from
Jul 7, 2023

Conversation

rocks6
Copy link
Contributor

@rocks6 rocks6 commented Apr 25, 2023

Background

#3079 - currently you have to either specify the ai_settings.yaml file or provide user input in order to set the ai name, roles, and goals. Ideally we can set these via command line and directly enter execution.

Changes

Added command line params for the above, and then passed them down to the configurator in a way that allows you to specify ANY of them (while prompting for the rest if necessary). Command line overrides will override any ai settings files if both are provided, by design.

Documentation

Updated comments and documented tests - TODO update main docs

Test Plan

Created new unit tests and manually ran a couple prompts

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

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Apr 25, 2023
@github-actions
Copy link
Contributor

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

@vercel
Copy link

vercel bot commented Apr 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview Apr 29, 2023 8:26pm

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Apr 26, 2023
@github-actions
Copy link
Contributor

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

@github-actions github-actions bot added size/xl and removed size/l labels Apr 26, 2023
@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

1 similar comment
@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@vercel vercel bot temporarily deployed to Preview April 26, 2023 21:04 Inactive
@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@vercel vercel bot temporarily deployed to Preview April 26, 2023 21:08 Inactive
@codecov
Copy link

codecov bot commented Apr 26, 2023

Codecov Report

Patch coverage: 61.62% and project coverage change: -0.02 ⚠️

Comparison is base (e4a337f) 50.05% compared to head (a3f3250) 50.04%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3250      +/-   ##
==========================================
- Coverage   50.05%   50.04%   -0.02%     
==========================================
  Files         116      116              
  Lines        4799     4826      +27     
  Branches      643      650       +7     
==========================================
+ Hits         2402     2415      +13     
- Misses       2218     2227       +9     
- Partials      179      184       +5     
Impacted Files Coverage Δ
autogpt/cli.py 0.00% <0.00%> (ø)
autogpt/config/ai_config.py 77.04% <ø> (-0.73%) ⬇️
autogpt/llm/chat.py 66.23% <0.00%> (-2.60%) ⬇️
autogpt/memory/vector/utils.py 78.12% <0.00%> (-5.21%) ⬇️
autogpt/prompts/prompt.py 41.50% <18.18%> (-4.15%) ⬇️
autogpt/llm/utils/__init__.py 49.25% <20.00%> (-1.48%) ⬇️
autogpt/main.py 23.68% <33.33%> (+1.60%) ⬆️
autogpt/agent/agent.py 59.18% <50.00%> (ø)
autogpt/setup.py 73.07% <73.33%> (-6.64%) ⬇️
autogpt/config/config.py 83.89% <93.33%> (+4.75%) ⬆️
... and 3 more

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

@rocks6
Copy link
Contributor Author

rocks6 commented Apr 27, 2023

merge messed this up, I need to fix it

@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@vercel vercel bot temporarily deployed to Preview April 27, 2023 20:28 Inactive
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Apr 28, 2023
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@vercel vercel bot temporarily deployed to Preview April 29, 2023 12:24 Inactive
@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@vercel vercel bot temporarily deployed to Preview April 29, 2023 12:25 Inactive
@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Apr 29, 2023
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

1 similar comment
@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jun 19, 2023
@github-actions
Copy link
Contributor

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

@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jun 21, 2023
@github-actions
Copy link
Contributor

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

@Pwuts Pwuts removed their assignment Jun 21, 2023
@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jun 24, 2023
@github-actions
Copy link
Contributor

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

@lc0rp lc0rp modified the milestones: v0.4.3 Release, v0.4.4 Release Jun 26, 2023
@lc0rp lc0rp modified the milestones: v0.4.4 Release, v0.4.5 Release Jul 6, 2023
@github-actions github-actions bot removed the conflicts Automatically applied to PRs with merge conflicts label Jul 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2023

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

@Pwuts Pwuts force-pushed the add-cmd-line-args branch from 98425e0 to fe4e161 Compare July 7, 2023 02:01
@Pwuts Pwuts modified the milestones: v0.4.5 Release, v0.4.4 Release Jul 7, 2023
Copy link
Member

@Pwuts Pwuts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Pwuts Pwuts merged commit 9e5492b into Significant-Gravitas:master Jul 7, 2023
dayofthedave pushed a commit to dayofthedave/Auto-GPT that referenced this pull request Jul 17, 2023
…avitas#3250)

* add capability to specify AI config at cmd line

* Make `--ai-goal` multi-parameter

* Fix argument forwarding in run.sh

---------

Co-authored-by: Reinier van der Leer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants