Skip to content

Support gym==0.10.8#324

Merged
toslunar merged 6 commits intochainer:masterfrom
muupan:gym_v0.10.8
Oct 9, 2018
Merged

Support gym==0.10.8#324
toslunar merged 6 commits intochainer:masterfrom
muupan:gym_v0.10.8

Conversation

@muupan
Copy link
Copy Markdown
Member

@muupan muupan commented Oct 4, 2018

This PR makes following changes to support gym==0.10.8.

  • remove calls of gym.undo_logger_setup
  • remove chainerrl/spaces.py
  • import gym.spaces explicitly
  • import gym.wrappers explicitly

Support of gym older than 0.9.7 is dropped.

Resolves #323, #325.

muupan added 5 commits October 4, 2018 18:16
- remove calls of gym.undo_logger_setup
- remove chainerrl/spaces.py
- import gym.spaces explicitly
- import gym.wrappers explicitly
since jupyter-console has been updated to use new prompt-toolkit
@muupan
Copy link
Copy Markdown
Member Author

muupan commented Oct 5, 2018

CI was broken by the update of jupyter-console but it's also fixed by this PR.

@muupan muupan changed the title [WIP] Support gym==0.10.8 Support gym==0.10.8 Oct 5, 2018
@muupan
Copy link
Copy Markdown
Member Author

muupan commented Oct 5, 2018

I confirmed that all the tests passed.

@toslunar toslunar self-assigned this Oct 8, 2018

from chainerrl import env
from chainerrl import spaces
from gym import spaces
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you fix the grouping of the imports?


from chainerrl import env
from chainerrl import spaces
from gym import spaces
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you fix the grouping of the imports?

@muupan
Copy link
Copy Markdown
Member Author

muupan commented Oct 9, 2018

I fixed the grouping of imports.

Copy link
Copy Markdown
Member

@toslunar toslunar left a comment

Choose a reason for hiding this comment

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

LGTM

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants