Skip to content

Conversation

xiaomaogy
Copy link
Contributor

No description provided.

@xiaomaogy xiaomaogy requested a review from mmattar October 9, 2018 23:24
```console
chmod +x <your_env>.x86_64
```
8. (Without Headless Mode) Start X Server and make the ubuntu use X Server for
Copy link

Choose a reason for hiding this comment

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

change to ... Start X Server and use it for display

You should receive a message confirming that the environment was loaded successfully.
8. Train the executable
10. Train the executable
Copy link

Choose a reason for hiding this comment

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

Run the executable OR Train your models

display:
```console
//Start the X Server, press Enter to come back to the command line
Copy link

Choose a reason for hiding this comment

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

isn't # more correct than //? See same for other lines.

5. Click Build to build the Unity environment executable.
6. Upload the executable to your EC2 instance within `ml-agents` folder.
7. Test the instance setup from Python using:
7. Change permission on your executable.
Copy link

Choose a reason for hiding this comment

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

Change the permissions of the executable.

3. Select Linux as the Target Platform, and x86_64 as the target architecture.
4. Check Headless Mode (If you haven't setup the X Server).
3. Select Linux as the Target Platform, and x86_64 as the target architecture
(the default x86 currently doesn't work).
Copy link

Choose a reason for hiding this comment

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

doesn't --> does not

4. Check Headless Mode (If you haven't setup the X Server).
3. Select Linux as the Target Platform, and x86_64 as the target architecture
(the default x86 currently doesn't work).
4. Check Headless Mode (If you haven't setup the X Server, if you don't use
Copy link

Choose a reason for hiding this comment

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

"Check Headless Mode (If you haven't setup the X Server, if you don't use Headless Mode, you have to setup the X Server to enable the training)" --> "Check Headless Mode if you have not setup the X Server. (If you do not use Headless Mode, you have to setup the X Server to enable training)"

training without the headless mode, you need to enable X Server on it. After
launching your EC2 instance using the ami and ssh into it, run the following
commands to enable it:
(Ubuntu)](https://aws.amazon.com/marketplace/pp/B077GCH38C). We've tested it
Copy link

Choose a reason for hiding this comment

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

Change to:

The AMI has been tested with p2.xlarge instance. Furthermore, if you want to train without headless mode, you need to enable X Server.

After launching your EC2 instance using the ami and ssh into it, run the following commands to enable it:

@xiaomaogy
Copy link
Contributor Author

Hi @mmattar, thanks for the comments! I've resolved all of them. I know the comments should be marked '#' in bash script, but in this doc '//' is maybe a little bit more clear than '#'.

@xiaomaogy xiaomaogy self-assigned this Oct 10, 2018
Copy link

@mmattar mmattar left a comment

Choose a reason for hiding this comment

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

Looks great. I have not tested this set-up myself though.

@xiaomaogy xiaomaogy merged commit 230e569 into develop Oct 11, 2018
@xiaomaogy xiaomaogy deleted the develop-aws-update branch October 11, 2018 00:43
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants