joshspeagle / dynesty

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

Fixes slice #269

Closed segasai closed 3 years ago

segasai commented 3 years ago

This is several commits. One that revises the pointvol setting and only sets if there is one single point (as otherwise that can sometimes overinflate the ellipsoid) I also remove the exception on slice sampling stuck (as discussed in #266) Add a longer/more extended pathology test.

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 880102601


Changes Missing Coverage Covered Lines Changed/Added Lines %
py/dynesty/dynamicsampler.py 2 3 66.67%
py/dynesty/sampling.py 23 24 95.83%
<!-- Total: 25 27 92.59% -->
Files with Coverage Reduction New Missed Lines %
py/dynesty/dynamicsampler.py 2 86.87%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 879968479: 0.04%
Covered Lines: 3387
Relevant Lines: 4707

💛 - Coveralls
joshspeagle commented 3 years ago

Looks good! Merging this in.