joshspeagle / dynesty

Dynamic Nested Sampling package for computing Bayesian posteriors and evidences
https://dynesty.readthedocs.io/
MIT License
347 stars 76 forks source link

remove all the direct calls of np.random.* #280

Closed segasai closed 3 years ago

segasai commented 3 years ago

218 incorrectly used np.random.directly instead of rstate

I've also added a test that will fail if it'll happen again.

joshspeagle commented 3 years ago

Excellent fix to both the tests and catching this bug, which would prevent full reproducibility.

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 919530575


Totals Coverage Status
Change from base Build 919359495: 0.0%
Covered Lines: 3595
Relevant Lines: 4662

💛 - Coveralls