Follow-up from "Allow any lal dictionary option and add numerical_relativity_file"
The following discussion from !896 (merged) should be addressed:
-
@colm.talbot started a discussion: (+1 comment) Thanks @cecilio.garcia-quiros! I think this is a good idea to include in
Bilby
.@gregory.ashton @moritz.huebner do you think we should suggest to users (probably in a post 1.1 release) that they not use, e.g.,
pn_spin_order
and usePNSpinOrder
instead so that this new method will catch those and we can remove the lines which explicitly look for those parameters?