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

Include seed to argument parser #197

Open
4 tasks done
tsirif opened this issue Dec 30, 2018 · 3 comments
Open
4 tasks done

Include seed to argument parser #197

tsirif opened this issue Dec 30, 2018 · 3 comments

Comments

@tsirif
Copy link
Collaborator

tsirif commented Dec 30, 2018

Hello there!

I would like to make a PR which would include the following things:

  • Add an cmd line argument for a seed.
  • Create a setter functions which takes a seed as an argument and reseeds relevant PRGs with it.
  • In exp.save save a "current" seed along with other stuff.
  • Modify "reload" routines to reseed properly.

Do you agree with these modifications? Where would they be most appropriate to include?

Also, what is the current status regarding the resumability of a specific named experiment?
Some optimizer objects are not stateless, are they also saved? and if they are not, should they?

Thank you!

@rdevon
Copy link
Owner

rdevon commented Jan 4, 2019

What is the high-level goal 2-4?

@tsirif
Copy link
Collaborator Author

tsirif commented Jan 5, 2019

To ensure that an experiment is reproducible with a specific seed, regardless if it was interrupted and resumed at some point or not.

@rdevon
Copy link
Owner

rdevon commented Jan 5, 2019

OK, let's see your PR then. Thanks for working on this!

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

No branches or pull requests

2 participants