-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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] support running older version tensorflow(version < 1.5.0) #3571
Conversation
Test PASSed. |
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.
We shouldn't clutter the example but the rest of changes look fine to me
Test FAILed. |
@richardliaw updated |
Test FAILed. |
jenkins, retest this please |
unrelated error @richardliaw |
Test FAILed. |
Could anybody help for this commit? @ericl @richardliaw |
Test failures look unrelated - sorry for the delay! |
support run rllib's examples with older version tensorflow(version < 1.5.0)
What do these changes do?
support run the rllib's examples with tensorflow (version < 1.5.0)
tf.boolean_mask only support tf.bool when tf version < 1.5.0
tf.variable_scope support auxiliary_name_scope when tf version >= 1.5.0
Related issue number
#3570