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

How to build RLDS on macOS #7

Open
jrabary opened this issue Oct 10, 2023 · 7 comments
Open

How to build RLDS on macOS #7

jrabary opened this issue Oct 10, 2023 · 7 comments

Comments

@jrabary
Copy link

jrabary commented Oct 10, 2023

Is there any documentation on how to install RLDS on macOS ?

@peterdavidfagan
Copy link

Hi @jrabary,

I was also interested in this but it doesn't look like there is a supported release on macOS (release file). The build files are in the source code but I guess it will take some work to parse through these and potentially make adjustments where needed; there might be fundamental challenges in supporting macOS.

@cemlyn007
Copy link

You can't install this because dm-reverb is only officially supported for Linux AMD64, but I do have a fork of dm-reverb where I did get it to compile to Mac ARM64. https://github.com/cemlyn007/reverb

@vyeevani
Copy link

The Bazel build has problems when using an unmodified version on a Mac. I ended up using a workaround where I just added this repo as a subtree to my work then pointed all my stuff at rlds.rlds allowing me to sidestep the Bazel install issues. Not the cleanest solution but the simplest :).

@peterdavidfagan
Copy link

peterdavidfagan commented May 28, 2024

Nice thanks @cemlyn007 I'll check out your fork next time I am looking to use rlds on MacOS.

@vyeevani do you have reference code for the above? Am I correct in my understanding, you are adding a submodule to your repo and then using relative imports that point towards the submodule files? Are there not issues related to the project dependencies (dm-reverb)?

@vyeevani
Copy link

I removed dm-reverb from the requirements. But your high level understanding is correct. If someone could update the bazel file to get that to work, it'd be greatly appreciated!

@vyeevani
Copy link

vyeevani commented May 28, 2024

@peterdavidfagan I ended up needing the bazel to do a better compile. My recipe is super janky though.

  1. build dm_reverb courtesy of @cemlyn007 's repo. I ended up having to do some other stuff on top of this to work with clang 15. You can find the exact changeset I used here: upgraded absl cemlyn007/reverb#1. The exact recipe that worked for me in terms of setting this up is:
conda create -n py39
conda activate py39
conda install python=3.9 -c conda-forge
pip install tensorflow-macos==2.16.1
  1. apply this patch.txt

this worked for me but no guarantees that this will work for everyone

@peterdavidfagan
Copy link

Thanks for this @vyeevani

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

4 participants