chainerrl icon indicating copy to clipboard operation
chainerrl copied to clipboard

Extend gym.Wrapper instead of env_modifiers

Open muupan opened this issue 7 years ago • 1 comments

Since gym has introduced its own interface to modify envs using gym.Wrapper, I think it is better to use it in ChainerRL instead of directly modifying methods as in env_modifiers.

muupan avatar Apr 24 '17 22:04 muupan

It seems gym.Wrapper assumes the base environment is a subclass of gym.Env. For example,

gym.wrappers.TimeLimit(chainerrl.envs.abc.ABC(), max_episode_steps=2)

raises AttributeError: 'ABC' object has no attribute 'metadata'.

toslunar avatar Nov 07 '17 08:11 toslunar