Default noise behavior for seed #74
Replies: 3 comments
-
|
Maybe a good solution would be to automatically set a new seed if it wasn't set already and pass that back to be included in the returned object? @AeRabelais Is the seed set in the catalogue init so should each class be handling them uniquely? |
Beta Was this translation helpful? Give feedback.
-
|
Okay for now my solution is to change the default behavior to be seed: int = None, so there still is an option to specify a seed but if they don't then it's random. |
Beta Was this translation helpful? Give feedback.
-
|
I'm starting another branch to work on the logging that saves the seed. @voetberg and @bnord , I'm wondering if we also want to save any additional info in the logging. I'm concerned that I'm mixing this train of thought up with the config file that will be generated after every run (the h5?). Here are some questions I have:
|
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
I'm working on the plotting tools for the pendulum and I realized that the noise function defaults to the random seed 42 each time we run it. This means that when we run it several times in a row, it will return identical output. Is this the default behavior we want or do we want it to use a new random seed each time and only use the seed=42 when specified?
I'm thinking about functionality in inference down the line where we might want to re-run multiple noise iterations and have it by default choose a new random seed. I'm also wondering how we want it to choose the random seed, because giving it a range of integers also seems restrictive.
I've attached a plot of what it would look like to by default draw the noise from a new random seed each time.

Beta Was this translation helpful? Give feedback.
All reactions