remove bounds internally before passing things off for dynamicsampler

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 bounds internally before passing things off for dynamicsampler #177

Closed joshspeagle closed 3 years ago

joshspeagle commented 4 years ago

There might be some small issues associated with bounds being copied internally when calling results and things like weight or stopping functions. I should change the internals slightly to avoid this happening when the results are never passed back to the user.

joshspeagle commented 4 years ago

This is now done on my local branch and just needs to get pushed upstream at some point. Looks like it doesn't break anything internally.

joshspeagle commented 3 years ago

I think this possible issue has actually been resolved due to how all the internal structural updates pushed by @segasai. Closing for now.