Using set.seed() - Githubissues

JGCRI / matilda

A probabilistic framework for the Hector simple climate model

Other

5 stars 3 forks source link

Using set.seed() #43

Closed bpbond closed 1 year ago

bpbond commented 1 year ago

Remind me please — let's discuss tomorrow @jk-brown

jk-brown commented 1 year ago

Will do. I have used set.seed() before. Are you thinking about this to the code chunks in the vignette?

bpbond commented 1 year ago

Basically, I just got a Hector error while building the vignette—I guess some random combination of parameters crashed the model (?). Anyway, that got me thinking:

We should double-check that the package is using try around Hector calls
The vignette should call set.seed before generating its parameter values, so we always know what's going to happen