-
Notifications
You must be signed in to change notification settings - Fork 9
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
Script results are not reproducible #83
Comments
Yes. lines 710 and 711 of instrument.py, random offsets. You would need to set a random seed |
Hi, @dkirkby @julienguy we face to this issue, or a related one, by looking at the reproducibility of quasar spectra with DLAs features. See desihub/desisim#386. Attempting to solve such issue I made a branch of specsim: test_repr_qquasar which modifies only the generate_random_noise function in simulator.py. What tests do you suggest to make to see if this affects other applications? or to corroborate such modifications makes sense? Thanks |
The This issue refers to random centroid offsets that are currently not reproducible. You can test if this is the source of the problem you are seeing by setting |
Running the following command twice gives similar but not identical results, which I wasn't expecting:
For example:
and
@julienguy Is this perhaps a "feature" of the new fastsim method calculating fiberloss? Do you use any random numbers?
The text was updated successfully, but these errors were encountered: