BUGFIX: make sure chain tracks all points (!1222) · Merge requests · lscsoft / bilby · GitLab

We will be having an extended maintenance of https://git.ligo.org, https://containers.ligo.org, and https://docs.ligo.org on 7 May 2024 starting at approximately 9am PDT and lasting for three to four hours. This is to start the process of syncing the data from the current instance to the re-engineered instance we have been working with GitLab on over the past few months. Starting at approximately 9am GitLab will be put into maintenance mode, meaning that all services will be in read-only mode. Throughout the maintenance there may be several periods of downtime. Please address any comments, concerns or questions to the helpdesk.

Colm Talbot requested to merge bugfix-act-sampling into master Feb 24, 2023

It looks like two bugs crept into !1187 (merged) just before it was merged.

one is the ordering of the updates of the bounding ellipses and the MCMC length for the LivePointMultiEllipsoidSampler. The fix here means that the scale attribute will update as expected within dynesty.
the ACT tracking broke because points weren't actually being added to the chain during the loop.

I also removed an excess full ACT calculation as nothing is bigger than inf!