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

[HANDS-ON BUG] Unit 7 on Apple Silicon #518

Closed
bpugnaire opened this issue Apr 21, 2024 · 2 comments
Closed

[HANDS-ON BUG] Unit 7 on Apple Silicon #518

bpugnaire opened this issue Apr 21, 2024 · 2 comments

Comments

@bpugnaire
Copy link
Contributor

Describe the bug

The proposed methodology to install ml-agents does not work by default on Mac

Material

Macbook Pro M2 running MacOs Ventura

To install you have to make sure you pull the right ml-agents branch ('develop') which config file are updated as demonstrated in this issue on the ml-agents repo Unity-Technologies/ml-agents#6019.

If you don't, you have endless issues of not being able to build ONNX wheel.

Then you have to test if the command ml-agents --help works but it can raise another error of an unidentified character which I have identified stem from this issue in grpc grpc/grpc#33714.
To fix it you need to run conda install grpcio since gRPC installed vi PYPI doesn't do well with conda venv it seems.

Lastly, its not much but I think you could provide the mac command to launch the training which is
mlagents-learn ./config/poca/SoccerTwos.yaml --env=./training-envs-executables/SoccerTwos/SoccerTwos.app --run-id="SoccerTwos" --no-graphics because if someone doesn't pay attention to the change in folder structure they enter more debugging since ml-agents doesn't like duplicated run-id and so on. Just a nice QOL change IMO

I can try to do a PR if you need.

Thank you for the great ressource nonetheless !

@simoninithomas
Copy link
Member

Hey there 👋 , yes if you can open a PR to add instructions, that would be useful for other students who have Apple Silicon CPU. Thanks 🤗

bpugnaire added a commit to bpugnaire/deep-rl-class that referenced this issue May 3, 2024
@simoninithomas
Copy link
Member

Hey there 👋 , I merged the PR thanks again for your contribution 🤗

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants