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

agent setup rm -rf <venv> ... should warn beforehand. #7404

Closed
1 task done
dagelf opened this issue Jul 12, 2024 · 3 comments · Fixed by #8417
Closed
1 task done

agent setup rm -rf <venv> ... should warn beforehand. #7404

dagelf opened this issue Jul 12, 2024 · 3 comments · Fixed by #8417
Labels

Comments

@dagelf
Copy link
Contributor

dagelf commented Jul 12, 2024

⚠️ Search for existing issues first ⚠️

  • I have searched the existing issues, and there is no existing issue for my problem

Which Operating System are you using?

Linux

Which version of AutoGPT are you using?

Latest Release

Do you use OpenAI GPT-3 or GPT-4?

GPT-4(32k)

Which area covers your issue best?

Installation and setup

Describe your issue.

Putting a file called setup.py in the main directory, is inviting people to just run it to see what happens... and if what happens is that their entire venv that might or might not be easily re-creatable just gets wiped, it will not create a good first experience...

Before such a command is run, it should ask for confirmation. Ie. either remove the -f, or explicitly prompt the user or point them in the right direction, or explain why.

Upload Activity Log Content

No response

Upload Error Log Content

No response

@dagelf dagelf changed the title setup.py rm -rf <venv> ... should warn beforehand. agent setup rm -rf <venv> ... should warn beforehand. Jul 12, 2024
@dagelf
Copy link
Contributor Author

dagelf commented Jul 12, 2024

I mean, anyone who reads the docs or setup.sh will see that pyenv and poetry or docker should be used... but not 100% of people who try this, will read it first.

At least a message in the script can explain to try to be more helpful.

Isn't the whole idea of autogpt to make computers more helpful?

Copy link
Contributor

github-actions bot commented Sep 3, 2024

This issue has automatically been marked as stale because it has not had any activity in the last 50 days. You can unstale it by commenting or removing the label. Otherwise, this issue will be closed in 10 days.

@github-actions github-actions bot added the Stale label Sep 3, 2024
Copy link
Contributor

This issue was closed automatically because it has been stale for 10 days with no activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment