AutoGPT
AutoGPT copied to clipboard
Add environment variable WORKSPACE_PATH
Background
I see the TODO when browsing code
# TODO: have this directory live outside the repository (e.g. in a user's
# home directory) and have it come in as a command line argument or part of
# the env file.
But it seems have competed 🤔
So I run it with command line argument, seems work

And then I saw workspace_path still can not set by ENV variable, so perhaps we can give it one.
I am still not sure the format of AI setting YAML file, so I not touch this part.
Changes
- Add environment variable
WORKSPACE_PATHto set workspace_path - Remove maybe redundant TODO
Documentation
Test Plan
- I have test locally about command line argument and .env file setting. Both success.
- Command line argument might have higher priority than environment variable.
- Test in other OS might be needed
PR Quality Checklist
- [v] My pull request is atomic and focuses on a single change.
- [v] I have thoroughly tested my changes with multiple different prompts.
- [v] I have considered potential risks and mitigations for my changes.
- [v] I have documented my changes clearly and comprehensively.
- [v] I have not snuck in any "extra" small tweaks changes
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| docs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | May 9, 2023 8:39am |
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 looks like a straightforward and simple change that might be a potential candidate PR to be reviewed prior to/during the ongoing re-arch effort.
Agree, this seems like a useful feature that shouldn't harm the re-arch too much.
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.
Thank you for submitting your contribution. However, it appears to be a duplicate of #1315. I will close this submission for now and recommend that you collaborate with that author if you are still interested in merging this PR.
Thank you for submitting your contribution. However, it appears to be a duplicate of #1315. I will close this submission for now and recommend that you collaborate with that author if you are still interested in merging this PR.
Many thanks for you help, and I am so sorry for my no timely response.