BUGFIX: dynesty - change npdim to ncdim

This MR will be needed to match an upstream change (https://github.com/joshspeagle/dynesty/pull/457) but I think is actually a bug in the current version. The axes that are returned are used to determine the number of "clustering" dimensions, however, we currently return an array with the total number of dimensions. This change will resolve that.

c.f. https://github.com/joshspeagle/dynesty/blob/master/py/dynesty/nestedsamplers.py#L567

We should probably also rebuild the conda release to add a pin on the dynesty version.

added <10 lines Bug labels

I've tested this with the latest master of dynesty and it works, whereas Bilby master does not.

changed the description

changed milestone to %2.2.1

requested review from @michael.williams

added 2 commits

• e841995b - 1 commit from branch master
• 0575d09d - BUGFIX: dynesty - change npdim to ncdim

Compare with previous version

@colm.talbot will the change impact sampling behaviour given we were using the number of dimensions instead of the number of clusters?

mentioned in merge request !1305 (merged)

added 6 commits

• 0575d09d...2f53006d - 5 commits from branch master
• bf64b1ae - BUGFIX: dynesty - change npdim to ncdim

Compare with previous version

resolved all threads

approved this merge request

approved this merge request

approved this merge request

added 2 commits

• 94bb56ce - 1 commit from branch master
• 25eecc0a - BUGFIX: dynesty - change npdim to ncdim

Compare with previous version

merged