TuringLang / AbstractMCMC.jl

Abstract types and interfaces for Markov chain Monte Carlo methods
https://turinglang.org/AbstractMCMC.jl
MIT License
87 stars 18 forks source link

Use `copy` instead of `deepcopy` for RNGs #147

Closed devmotion closed 3 weeks ago

devmotion commented 3 weeks ago

My take-away from https://github.com/JuliaLang/julia/issues/42899 is that we should use copy instead of deepcopy for RNGs. More generally, based on https://github.com/JuliaLang/julia/issues/42899#issuecomment-1058487002 we might also want to use copy for models and samplers (possibly requires defining copy in a few downstream packages).

sunxd3 commented 3 weeks ago

interesting, thanks David