RobertTLange / gymnax

RL Environments in JAX 🌍
Apache License 2.0
577 stars 54 forks source link

Refactoring #68

Closed Aidandos closed 3 months ago

Aidandos commented 3 months ago
  1. I fixed most of the typing issues under pylint --strict. 1.1 This required unifying the interface of step, step_env, get_obs, 'is_terminal` for proper inheritance.

  2. spaces now depends on gymnasium instead of gym, adding an additional dependency. Maybe later on gym dependency can be completely removed since it's not maintained anymore? haven't looked into that.

  3. I removed _DeviceArray from the tests as it's deprecated. I use jax version 0.4.24

  4. I ran everything in python 3.10 so technically you could support that too now.

  5. In wrapper/brax.py, GymnaxtoBraxWrapper has two new methods backend and observation_size, as its required to be defined under newest brax version. I used brax version 0.10.0.