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

TimeLimit refactor with Monitor Simplification #482

Merged
merged 21 commits into from Feb 1, 2017

Conversation

Projects
None yet
3 participants
@nottombrown
Contributor

nottombrown commented Feb 1, 2017

This merge in a rebased version of the unstable branch

Includes the following:

  • Default to autowrapping all non-universe envs in TimeLimit wrapper
  • Add max_episode_steps
  • A refactor of the Monitor
  • Include timestep_limit for backwards compat
  • Add play for human debugging
@nottombrown

This comment has been minimized.

Show comment
Hide comment
@nottombrown

nottombrown Feb 1, 2017

Contributor

Received in person LGTM from @joschu, so merging in now 🎉

Contributor

nottombrown commented Feb 1, 2017

Received in person LGTM from @joschu, so merging in now 🎉

@nottombrown nottombrown merged commit d337f4e into master Feb 1, 2017

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details

@nottombrown nottombrown deleted the unstable-rebased branch Feb 1, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment