Skip to content

Make sure to include correct message in ValueError

Serguei Ossokine requested to merge fix_error_msg into main

Previously when f_ref>max frequency in inspiral, the waveform would correctly raise ValueError and print the Input domain error message, however, it was missing from the actual ValueError msg. Thus it was not correctly caught by bilby during sampling. This commit adds the msg, both for aligned-spin and precessing models. Also adds a more helpful message for the case where things do truly fail and the sampling should be stopped.

Merge request reports

Loading