-
Notifications
You must be signed in to change notification settings - Fork 7
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
Could not find a version that satisfies the requirement tensorflow<1.16,>=1.15 #39
Comments
You should be able to run most of the code without MuJoCo, you can just change I doubt it'll work with TensorFlow 2 though, perhaps if you put it into v1 compatibility mode. Eventually I hope to port to TFv2 or PyTorch, but that's a big job and isn't a priority right now. Try Python 3.7 -- I think there are pip wheels for TensorFlow 1.15 for that. |
@AdamGleave Porting the code to TensorFlow 2 should not take that much time, given there are already scripts that automate this process to some extent, so I think you should really do it (if you have some time). I will see what I can do meanwhile. |
"Should" being the key word there. One major complication is that the RL library we use, Stable Baselines, is TFv1 or PyTorch (but no TFv2). I'd welcome a PR starting on a port, but I doubt I'll have the bandwidth to do this myself in the near future. |
@AdamGleave Is it possible to use |
I'm OK with switching to |
Indeed, it doesn't work with Is it possible to know if you then plan to make this package use |
I'm not planning on switching to Right now my priority is on adding some new experiments, and the codebase works OK for me (Python 3.7, Linux, TensorFlow 1.15). I will want to eventually clean it up and make it as usable as possible for a broader audience, but I won't be prioritizing that until the features are frozen. |
A workaround you could try is to use the Docker image which should avoid these dependency issues. |
Related: hill-a/stable-baselines#1012 (comment) "If you (or others) want to support and maintain a TF2 version, we would be also happy about it. But I have to warn you that it is quite time consuming." |
This problem may not be related to this specific package, but it can happen if you install this package, so I am opening the issue here too.
When I try to install this package with
pip
(version 19.0.3 with Python 3.8), I get the errorI am using a 64-bit version of Python on a Mac OS Catalina (10.15.7). Meanwhile, this problem seems to be temporarily solved by changing the specific version of TensorFlow that is installed in the
requirement.txt
file totensorflow==2.3.0
. However, I have not confirmed that this problem is solved because I was not yet able to run any code in this repo, given that we apparently also need mojuco as a requirement for this package, but mojuco requires a license, and that's quite annoying.The text was updated successfully, but these errors were encountered: