Resolve #222 Inconsistent use of `priors` and `prior`
This renames all the cases of "prior" to "priors" in the gravitatinal wave likelihood.
Closes #222 (closed)
Merge request reports
Activity
I've resolved this by fixing on "priors" rather than "prior". This will break scripts which use marginalization. Sorry.
The only way to avoid breaking those scripts is to accept both
prior
andpriors
. In the near-term this may save a few headaches, but in the long-term I think we'll look at that and say that is crazy to accept two names for the same thing..The same thing applies if we change "priors" to "prior", but this will break all scripts, which is ultimately why I opted for only breaking marginalization scripts.
added 1 commit
- 704e7c40 - Resolve #222 (closed) Inconsistent use of `priors` and `prior`
mentioned in commit 791681c4
Please register or sign in to reply