Make likelihood condition strictly increasing (!846) · Merge requests · lscsoft / bilby · GitLab

Snippets Groups Projects

Maintenance will be performed on git.ligo.org, containers.ligo.org, and docs.ligo.org on Tuesday 22 April 2025 starting at approximately 9am PDT. It is expected to take around 30 minutes and there will be several periods of downtime throughout the maintenance. Please address any comments, concerns, or questions to the helpdesk. This maintenance will be upgrading the GitLab database in order to be ready for the migration.

Merged Colm Talbot requested to merge dynesty-likelihood-condition-fix into master 4 years ago

There's a bug in the acceptance condition used in dynesty, see https://github.com/joshspeagle/dynesty/pull/203.

This fixes our walk function.

Approved by

Activity

Colm Talbot added Bug High priority Sampling labels 4 years ago

added Bug High priority Sampling labels
Sylvia Biscoveanu approved this merge request 4 years ago

approved this merge request
Gregory Ashton approved this merge request 4 years ago

approved this merge request
Gregory Ashton @gregory.ashton · 4 years ago

Maintainer

@colm.talbot do we also need the dynesty release for this to resolve the issue? Or is this sufficient?
Gregory Ashton merged 4 years ago

merged
Gregory Ashton mentioned in commit 308c8219 4 years ago

mentioned in commit 308c8219

Please register or sign in to reply