Skip to content

add options to lalapps_splitSFTs for start/end times

David Keitel requested to merge david-keitel/lalsuite:splitSFTs-start-end-times into master

Description

Adds new options --minStartTime and --maxStartTime (-ts/-te) to lalapps_splitSFTs, which should come in handy when dealing with very large (long-duration plus broad-band) merged input files and wanting to parallelize the main frequency-splitting step. The outputs of those parallel jobs can then be easily recombined by plain cat. Or the new options could be used just for time-extraction for its own purpose, obviously.

API Changes and Justification

@duncanmmacleod I've never been quite clear on this: is adding/changing commandline arguments of executables considered equivalent to API additions/changes? (Hence 2nd checkbox in this case?)

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

I've added the new options to the test script (though not all possible permutations). I'll keep it on "WIP" until @rodrigo.tenorio gets a chance to test this in practice. But comments from @karl-wette would already be appreciated.

Edited by David Keitel

Merge request reports