amices / mice

Multivariate Imputation by Chained Equations
https://amices.org/mice/
GNU General Public License v2.0
433 stars 107 forks source link

use x$lastSeedValue, not .Random.seed (which may not exist) #502

Closed MichaelChirico closed 1 year ago

MichaelChirico commented 1 year ago

I'm seeing an error where cbind() wounds up run without .Random.seed set, which this solves.

Per ?.Random.seed, no seed is set initially in the session until the first time set.seed() is run:

 Initially, there is no seed; a new one is created from the current time and the process ID when one is required.

I don't know enough about the package to know if this fix is for sure correct, but I noticed a similar fix applied in the same file:

https://github.com/amices/mice/commit/1189fe10aee92077f05d68279da7974090e86d60

stefvanbuuren commented 1 year ago

Thanks. We intend to tackle this one with #515.

stefvanbuuren commented 1 year ago

Adressed by c084cdd48