-
Notifications
You must be signed in to change notification settings - Fork 312
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
Failed building wheel for dm-tree #1442
Comments
It seems that dm-tree is not compatible with Python 3.8: google-deepmind/dm_control#135 |
Great catch! The best way for us to specify this is to update setup.py In particular, you'll want why don't you give it a shot? |
(you should also leave a comment explaining 3.8 is unsupported, with a link to the issue you found) |
The issue is that compiling dm-tree from source requires installing bazel. So it's not that we are not compatible with python 3.8, but that on 3.8, we have an extra system dependency. Do you think we should still restrict our supported range of python versions for that? |
I’m happy to modify the setup.py either way. It’s be great to support 3.8 by adding the bazel dependency, but on the other hand we already have a lot of dependencies, and the bazel binary is rather large (~120mb). Let me know which way to go. |
The root of the problem is not bazel, but DeepMind failing to upload a pre-compiled wheel for Python 3.8 to PyPI, causing pip to revert to trying to compile some native extensions from source code (which requires bazel). We should probably just wait for them to upload the wheel. Until then, we should at least properly declare that not all configurations support Python 3.8. |
This should be fixed -- check again? |
Yep. Looks like it's working now |
Hey, I have the same issue with Python 3.6.9.. Anyone has a fix already? |
any update? |
Same issue here. I have Python 3.6.8 |
oh,damn. same problem in 3.12 when I use pip install keras. Where is the solution? |
Upgrading to Python 3.11.5 solved the problem for me. |
success right now amazingly while I still do not know what problem it happened. |
Installing garage dependencies into a fresh conda environment (Python 3.8) via
pip install --no-cache -e ."[mujoco,dm_control]"
yields the following errors related to dm-tree. These may or may not be specific to my computer/macOS, but they are reproducible in every fresh conda env that I make. If instead I try to install from PyPI (pip install --no-cache -e garage"[mujoco,dm_control]"
), it saysPackages installed from PyPI cannot depend on packages which are not also hosted on PyPI
since dm_control is from deepmind's repo.The text was updated successfully, but these errors were encountered: