Skip to content

SimInspiralFD frequency spacing

Colm Talbot requested to merge colm.talbot/lalsuite:siminspiralfd-frequency-spacing into master

Description

Closes #513 (closed). See the discussion in that issue for the details.

TL;DR This MR makes the returned frequency series for lalsimulation.SimInspiralFD have the requested frequency spacing.

cc @geraint.pratten @maria.haney @aaron.zimmerman @patricia-schmidt @jolien-creighton

I'm unfamiliar with lalsuite development methods. Is there some kind of unit testing that this can be added to?

API Changes and Justification

Backwards Compatible Changes

  • This change introduces no API changes
  • This change adds new API calls

Backwards Incompatible Changes

  • This change modifies an existing API
  • This change removes an existing API

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

Please provide details on any reviews related to this change and and the associated reviewers.

Edited by Colm Talbot

Merge request reports