confusing rwalk messages for all dynesty_sample options
@colm.talbot : Taking the jump from bilby < 2 to > 2, several of us got very confused by the message that is printed at the start regarding the chosen dynesty sampling mode.
As can be seen in the 30ish lines starting from https://git.ligo.org/lscsoft/bilby/-/blob/master/bilby/core/sampler/dynesty.py#L447 it always prints Using the bilby-implemented rwalk
for all 3 choices of rwalk
, acceptance-walk
and act-walk
. The rest of the messages actually differs and correctly lists the relevant parameters for each method. But we got quite worried that there could be a bug making us run rwalk
instead of acceptance-walk
as we wanted to, and we had to go through the rabbit hole of finding this code snippet to understand what's going on.
To resolve the confusion, should there simply be a {sample}
instead of rwalk
in L449, L463, L471...?