Maintenance work for compatibility with the OpenAI gym's latest version. about gym_tcp_api HOT 6 CLOSED

@zoq @KimSangYeon-DGU I suppose that the newer gym versions are kind of python3 supporting.
env =gym.make(en) File "/usr/lib/python2.7/site-packages/gym/envs/registration.py", line 171, in make return registry.make(id, **kwargs) File "/usr/lib/python2.7/site-packages/gym/envs/registration.py", line 123, in make env = spec.make(**kwargs) File "/usr/lib/python2.7/site-packages/gym/envs/registration.py", line 87, in make env = cls(**_kwargs) File "/usr/lib/python2.7/site-packages/gym/envs/toy_text/taxi.py", line 104, in __init__ discrete.DiscreteEnv.__init__(self, nS, nA, P, isd) File "/usr/lib/python2.7/site-packages/gym/envs/toy_text/discrete.py", line 38, in __init__ self.action_space = spaces.Discrete(self.nA) File "/usr/lib/python2.7/site-packages/gym/spaces/discrete.py", line 15, in __init__ super().__init__((), np.int64) TypeError: super() takes at least 1 argument (0 given)

The constructor issue is inherent trouble of Python2 code when executed using Python3.
Also the higher versions of gym is written in Python3.
Should we move to Python3?

from gym_tcp_api.

@MuLx10, Right, the problem was resolved, just by running the server in default python3.

1. Edit from gym.wrappers.monitoring import Monitor to from gym.wrappers import Monitor in priv/worker.py.
2. Just run the server in python3.

After checking it, feel free to make a pull request. Thanks for taking time to do. :)

p.s. It would be a good idea to make a note indicating that the version of python should be 3 for compatibility with the latest gym for users.

from gym_tcp_api.

Hey, @KimSangYeon-DGU and @zoq I want to contribute. I see that the Monitor doesn't seem to work for gym>0.9.5.

from gym_tcp_api.

@MuLx10, Sure, go ahead. :)

from gym_tcp_api.

@KimSangYeon-DGU @zoq I have made the necessary changes. Please review it (#13).
Thank You

from gym_tcp_api.

Thanks again!! @MuLx10

from gym_tcp_api.