-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
[bazel] upgrade cython package #45436
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
seems that I need to stick with 0.29.x |
d647680
to
eb8b608
Compare
to latest release 3.0.10 Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
b98d466
to
301e723
Compare
@@ -183,7 +183,7 @@ | |||
subprocess.run( | |||
"sudo apt-get install -y build-essential curl unzip psmisc".split(" ") | |||
) | |||
subprocess.run("pip install cython==0.29.26 pytest".split(" ")) | |||
subprocess.run("pip install cython==0.29.37 pytest".split(" ")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sven1977 , do the rllib people still use this bisect tool?
fwiw, we have repo wide bisect tool now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping for review. |
Signed-off-by: Lonnie Liu <lonnie@anyscale.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this!!
ok I am merging. @sven1977 (or any one else), if you need to revert this or some files in this, let me know. |
to 0.29.37; required for bazel upgrade.