[Proposal] Add wrapper checker

Farama-Foundation / Gymnasium

An API standard for single-agent reinforcement learning environments, with popular reference environments and related utilities (formerly Gym)

https://gymnasium.farama.org

MIT License

7.29k stars 810 forks source link

[Proposal] Add wrapper checker #6

Open pseudo-rnd-thoughts opened 2 years ago

pseudo-rnd-thoughts commented 2 years ago

In v24 / 25, we added an environment checker for users to check that environment follow the API correctly. As some environments implement their own wrappers and to check our own, it would be helpful to add a wrapper checker.

Proposed checks

Check that all observations and actions spaces are valid (all observations are contained in space)
Check the input and output types for reset and step

Kallinteris-Andreas commented 1 year ago

is this passiveEnvChecker? close?

pseudo-rnd-thoughts commented 1 year ago

No, the passive environment checker is for the environment. I was more imagining a function similar to 'check_env' but for a wrapper

noahfarr commented 9 months ago

Is this still an open issue? If so, I'd like to give it a shot.

pseudo-rnd-thoughts commented 9 months ago

@noahfarr Hey, yeah, I don't think that anyone has worked on this so go for it