You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I added a script with commit 959f4fc that tests reproducibility expectations as formulated with PR #528 (and see documentation and code comments of set_all_rngs().
I think we should clarify what the expected behavior of weather among gridcells in gridded mode if random seed is 0 should be (a seed of 0 signals non-reproducible random number sequences):
Should weather be identical among cells (for a given run) in gridded mode even if random seed is 0?, or
Should weather vary among cells in gridded mode if random seed is 0? [The script currently tests for this behavior.]
Either way, the current implementation fails because some gridcells produce identical weather and other gridcells produce (awkwardly [see below]) different weather with a random seed of 0.
I tested this on branch "Seed_Dispersal" (commit b995a9d; June 22, 2023) and on branch "master" (commit 04fb112; June 20, 2023).
tools/check_STEPWAT2.sh
#-----------------------------------------------------------------------
#--- * if seed == 0 (output cannot be reproduced among runs) ------
#- Set model.in: 2 100 0 (niter, nyrs, seed)
#--- ** weather is different among cells ------
[1] FALSE FALSE TRUE
or
#-----------------------------------------------------------------------
#--- * if seed == 0 (output cannot be reproduced among runs) ------
#- Set model.in: 2 100 0 (niter, nyrs, seed)
#--- ** weather is different among cells ------
[1] FALSE TRUE TRUE
*awkwardly different
The text was updated successfully, but these errors were encountered:
I added a script with commit 959f4fc that tests reproducibility expectations as formulated with PR #528 (and see documentation and code comments of
set_all_rngs()
.I think we should clarify what the expected behavior of weather among gridcells in gridded mode if random seed is 0 should be (a seed of 0 signals non-reproducible random number sequences):
Either way, the current implementation fails because some gridcells produce identical weather and other gridcells produce (awkwardly [see below]) different weather with a random seed of 0.
I tested this on branch "Seed_Dispersal" (commit b995a9d; June 22, 2023) and on branch "master" (commit 04fb112; June 20, 2023).
or
*awkwardly different
The text was updated successfully, but these errors were encountered: