ppasupat commented 1 year ago

Description

For all registered environments (excluding FlightWoB), added tests for the following:

Run gymnasium.utils.env_checker.check_env, which checks the observation space and seed determinism (except click-pie, where the UI position can jitter by a few pixels).
Verify the flattened observation space.

Also added a utility for inspecting the observation space. Tasks were modified to pass the tests.

Addressed part of issue #15.

Type of change

Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

[x] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (not applicable)
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

ppasupat commented 1 year ago

Sorry for not responding earlier. I have addressed the comments.

pseudo-rnd-thoughts commented 1 year ago

Thanks for the PR, looks good