Making all tests pass with Implement_for functionality. Enables support most gym/gymnasium versions

vikashplus commented 9 months ago

FEATURE: Adding support with passing tests for multiple gym/gymnasium versions

adding a helper util to prioritize gymnasium import over gym
standardizing gym_registry_specs for robohive env registration
moving all self.np_random.randint to self.np_random.choice as its more standard across versions
bugfix in range specifications. high-low<0 for a few instances
Adding a buffer (*_) in step and forward to accommodate different number of entities returned
starting to use env.unwrapped instead of env.env.....
robustifying test_logger. It can effectively fish out the log names now

vikashplus commented 9 months ago

@vmoens @Vittorio-Caggiano @0wu -- Can I get a round of reviews on this features? Its largely starts from @vmoens ideas and initial commit on how to make the repo compatible with most gym/gymnasium versions, and builds forward from there. Big Kudos to Vincent for leading and helping me landing this feature.

vikashplus commented 9 months ago

@vmoens -- I tried something like this - https://github.com/vikashplus/robohive/blob/implement_for/robohive/tests/test_versions.sh All the tests succeed.

Any other salient versions we should check for?

vikashplus / robohive

Making all tests pass with Implement_for functionality. Enables support most gym/gymnasium versions #119