search

joshspeagle / dynesty

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

Rebase of the Github actions/tests PR #231

Closed segasai closed 3 years ago

segasai commented 3 years ago

I've rebased my github actions PR to avoid merging issues here https://github.com/joshspeagle/dynesty/pull/229

(I've just git cherry-picked all the commits to this branch)

joshspeagle commented 3 years ago

Okay, I've gone through all the changes and this looks good to merge in. Thanks for moving all this around and also for adding in the new test case. This should make future changes both easier and more robust, and also enable better coverage testing.

I'll look into updating the docs, author credits, and cutting a new public release (v1.1) by the end of the week.

segasai commented 3 years ago

Thanks. And yes, I think looking at test coverage should help seeing which parts of the code are not exercised by the tests, and eventually add more of them. ATM (I think the test code coverage was 65%)