Skip to content

BUGFIX: dynesty - change npdim to ncdim

Colm Talbot requested to merge remove-npdim into master

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.

Edited by Colm Talbot

Merge request reports

Loading