-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
Porting openai/gym's foundation classes to Gym.jl #5
Comments
That's a good idea. A structured way to represent observation and action space is currently required. |
Cool. I'll start working on it right away. |
I wanted to know if for certain action spaces we should switch to 0-based indexing or stick with 1-based indexing. For example, should Discrete spaces begin with 0 or with 1? |
Beginning with 1 would make it more julia-like. |
Closing Issue! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi. I just wanted to get an opinion on porting openai/gym's foundational classes (
Space
,Env
, etc) to Gym.jl and then building additional environments atop these base classes. My reasoning is that for someone who is coming from using openai/gym, transitioning to Gym.jl will be effortless, because they'd already be familiar with underlying structure of the package.The text was updated successfully, but these errors were encountered: