-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
[RLlib][Tune] Build Ray-ML docker image for Python 3.9 #16599
Comments
Hmm can we just not build Ray-ML Docker images for 3.9 for now? The standard Ray docker images shouldn’t need requirements_rllib and requirements_tune |
Also what’s the exact error you are getting? another alternative is to just skip over the libraries that don’t have 3.9 support- something like this https://stackoverflow.com/questions/22250483/stop-pip-from-failing-on-single-package-when-installing-with-requirements-txt |
If it is ok with others to not build Ray-ML Docker images for 3.9, that can definitely be a solution. |
cc @richardliaw do we have a target on when to fully support Python 3.9? |
What is the problem?
Currently building Ray docker image with Python 3.9 times out, because some of the direct and transitive dependencies pinned in
python/requirements/rllib/requirements_rllib.txt
andpython/requirements/tune/requirements_tune.txt
do not support Python 3.9, and the pip dependency resolver goes into an edge case.Ray version and other system information (Python version, TensorFlow version, OS):
Python 3.9, Ray 1.4.1
Reproduction (REQUIRED)
https://github.com/ray-project/ray/pull/16571/files
The text was updated successfully, but these errors were encountered: