-
Notifications
You must be signed in to change notification settings - Fork 237
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
custom gym environments are not compatible with current gym API #29
Comments
here is a minimal reproduction - https://github.com/MatthewCaseres/gdrl_reproduction/tree/main |
Can you pull the image instead of building it from scratch and check the gym version? I'm aware they have changed the gym package considerably in recent months. I will be updating the codebase with the second edition of the book. |
I feel that this issue should not have been closed so quickly? I assumed that since no version of |
Correct. Let me try pushing the right versions soon. |
|
I think you are depending on an older version of gym in some of the environments you made, a common error is
walk_env.py
has step function returnreturn (int(s), r, d, {"prob": p})
but the version of gym I'm on (0.26.0) seems to wrap your environment in some time limit andgym.make('RandomWalk-v0')
returns something like this<TimeLimit<OrderEnforcing<PassiveEnvChecker<WalkEnv<RandomWalk-v0>>>>>
.Your dockerfile doesn't pin gym to any specific version so I don't know what the right version is.
The text was updated successfully, but these errors were encountered: