Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updates for basic support of Gymnasium (+ updates to rendering approach) #68

Open
wants to merge 4 commits into
base: default
Choose a base branch
from

Conversation

SimonOuellette35
Copy link

@SimonOuellette35 SimonOuellette35 commented May 24, 2023

  • if you're interested, here is a pull request that provides basic support for Gymnasium. I had to modify how rendering works by leveraging the HumanRendering wrapper in Gymnasium, which in turn forced me to somewhat redesign how selection of rendering mode works (see updates to the README)
  • Caveat 1: this probably breaks pre-gymnasium code (haven't tested it)
  • Caveat 2: 'raw' rendering mode no longer supported.

@rhaps0dy
Copy link

You need to change from gym.envs.registration import register to from gymnasium.envs.registration import register in __init__.py.

reverted the temporary README changes I had made for my fork
@SimonOuellette35
Copy link
Author

You need to change from gym.envs.registration import register to from gymnasium.envs.registration import register in __init__.py.

Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants