Skip to content

SynthesizeCWDraws: revert to consistent (h0,cosi) output in --outputInjParams

David Keitel requested to merge david-keitel/lalsuite:synthCW-h0-cosi into master

Description

This fixes a small issue introduced last year by !482 (merged) which I only just noticed: for the SynthesizeCWDraws module and the three lalapps codes calling it, the internal conversion (h0,cosi)<->(Aplus,Across) is fine as far as I can tell, but the file output should have been left as (h0,cosi) for backwards compatibility and since this was still what the columns header string said. It's also still the more natural output tuple here since the input arguments for the lalapps codes are all --h0 --cosi only, anyway.

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

Small fix for @karl-wette to review. For the record, it's unlikely that this would have been caught by a test script, since its a header-content mismatch in a secondary output file which is usually only looked at for debugging.

Merge request reports