Skip to content

Conversation

@Emily0219
Copy link
Contributor

What do these changes do?

changes about action space in different situations.

Related issue number

#1972

changes about action space
@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/5250/
Test PASSed.

@Emily0219 Emily0219 changed the title Update bc/policy.py [rllib]Update bc/policy.py May 7, 2018
Copy link
Contributor

@richardliaw richardliaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks fine!

@richardliaw
Copy link
Contributor

@Emily0219 would it be possible to fix the flake/style issues?

You can find the issues by running flake8 . in this directory: ray/python/ray/rllib/bc

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/5387/
Test PASSed.

@richardliaw
Copy link
Contributor

test failures unrelated. @Emily0219 thanks for contributing!

@richardliaw richardliaw merged commit 79b45c6 into ray-project:master May 15, 2018
alok added a commit to alok/ray that referenced this pull request May 21, 2018
* master:
  Pin Pandas version for Travis to 0.22 (ray-project#2075)
  Fix python linting (ray-project#2076)
  [xray] Fix GCS table prefixes (ray-project#2065)
  Some tests for _submit API. (ray-project#2062)
  [rllib] Queue lib for python 2.7 (ray-project#2057)
  [autoscaler] Remove faulty assert that breaks during downscaling, pull configs from env (ray-project#2006)
  [DataFrame] Refactor indexers and implement setitem (ray-project#2020)
  [rllib]Update bc/policy.py (ray-project#2012)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants