-
Notifications
You must be signed in to change notification settings - Fork 4.4k
update to python instructions #530
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
Conversation
added link to download python and indicated the dependencies can be installed with pip3 in the next step
docs/Installation.md
Outdated
|
|
||
| In order to use ML-Agents, you need Python 3 along with | ||
| the dependencies listed in the [requirements file](../python/requirements.txt). | ||
| the dependencies listed in the [requirements file](../python/requirements.txt). You can install these dependencies using pip3 in the next step. Additionally, you can download and install Python 3 from [here](https://www.python.org/downloads/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure its a good idea to start discussing specific installations outside the subsections below. Please fold this info into both the Windows guide and the Mac/Unix users section below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok i moved it down
docs/Installation.md
Outdated
|
|
||
| In order to use ML-Agents, you need Python 3 along with | ||
| the dependencies listed in the [requirements file](../python/requirements.txt). | ||
| the dependencies listed in the [requirements file](../python/requirements.txt). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove extra space at end.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
docs/Installation.md
Outdated
|
|
||
| ### Mac and Unix Users | ||
|
|
||
| If you do not have Python, you can download and install Python 3 from [here](https://www.python.org/downloads/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd slightly re-word to be more inline with our other doc: Download and install Python 3 if you do not already have it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
added link to download python and indicated the dependencies can be installed with pip3 in the next step