Skip to content

Added AMR bounds check for Mchirp-q coordinates

Daniel Wysocki requested to merge daniel.wysocki/rapidpe:Mchirp_q_bounds_check into master

This addresses #7 (closed) by adding a missing bounds check when working in \mathcal{M}_c-q coordinates. To avoid missing bounds checks for any future supported coordinate systems, a NotImplementedError is added in the event of an unrecognized coordinate system, to force us to add the bounds check.

Also makes some minor improvements to the surrounding code: avoiding the creation of extra arrays, inconsistent whitespace, unnecessary newline escaping.

Merge request reports