dfm / emcee / 29197389920

96%

push

github

fix: avoid mutating sampler.pool as a side effect of pickling (#572)

EnsembleSampler.__getstate__ assigned self.__dict__ to a local name
and then set pool to None on it, which mutated the live object in
place. Pickling a sampler (directly, via copy.copy, or via any
library that calls __getstate__) permanently discarded its pool and
broke subsequent parallel sampling on the original instance.

Signed-off-by: Mike German <mike@stepsventures.com>

279 of 324 branches covered (86.11%)

9 of 10 new or added lines in 2 files covered. (90.0%)

1625 of 1689 relevant lines covered (96.21%)

3.43 hits per line