Use 'withr::local_seed()' instead of 'set.seed()' in starting board functions

piecepackr / ppgames

R package with piecepack game rules, parsers, and visualization tools

https://trevorldavis.com/R/ppgames

Creative Commons Attribution Share Alike 4.0 International

5 stars 0 forks source link

Closed trevorld closed 3 years ago

trevorld commented 3 years ago

withr::local_seed() should prevent possible side effects of the seed state being changed
Keeping setting seed within the function makes it easier to indicate reproducible random starting boards via PPN
If seed argument is NULL perhaps don't reset to NULL? So we could set seed outside the function if desired.