-
Notifications
You must be signed in to change notification settings - Fork 93
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
Random seeding of environments would be cool #21
Comments
Thanks for the suggestion! We're getting back to making some updates to the library and will get this in place. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, first, thank you for your work in putting this together! Good stuff.
I wanted to make a suggestion, that there could be a lot of value in adding random seeds across the board to your environments. This feature would enables users to a) focus on solving a single instance (e.g., a large/hard one where numerical optimization fails) and b) reproduce results and, by virtue of this, benchmark policies. This is alluded to in this merge, and I think the work is basically the same for the other models. I didn't see a way to do this via a PR given my own time limitations so just wanted to drop the thought here.
Thanks!
The text was updated successfully, but these errors were encountered: