-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
Checklist for maturity #21
Comments
@elliottower For "Support serialization via Pickle/EzPickle", you mean for serializing envs? |
Yeah that’s for environments, but if it’s not possible here due to the complex emulators it’s probably not a big deal |
@elliottower For explicit versioning, does shimmy plan to use that? looking at the repo it's still using the 3 number versions |
Oh sorry I think there’s a misunderstanding. Explicit versioning is for environments so like whenever you change the logic or anything that would make training go differently, change it from pong_v0 to v1, etc. For emulated envs like here I would think it would be more similar to shimmy having DmControlCompatibilityWrapperV0 for example, which gets updated when it loads the envs differently or would somehow otherwise affect training like rewards changed etc. See the farama.org project standards for more info |
@elliottower So the checklist is progressing, there is two remaining items I have questions with: |
Tagged you in a discord thread with the web dev guy manuel and a thread in all devs about farama notifications |
Checklist for Maturity
Copied from shimmy a few months ago, feel free to mark things as complete (I believe anyone can)
The text was updated successfully, but these errors were encountered: