Skip to content

Add spectral coherence tool for Fscan

Evan Goetz requested to merge evan-goetz/lalsuite:add-spec-coh into master

Description

This merge request adds a new lalapps program called lalapps_spec_coherence that computes the coherence between two different sets of SFTs. It will be used for the Fscan data quality monitor. This program would overhaul some of the calculation work done in lalapps_Fscan_coherenceFromSFTs.py but not the plotting. The code is similar to spec_avg_long.c but with specifics to account for two sets of SFTs and to compute coherence and not averaged spectra.

A unit test has been added to test this new program as well, though it is very simple since the coherence of SFTs are computed against itself so the output values are all ones.

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

@karl-wette or @david-keitel would be appropriate reviewers since they've likely peeked at the spec_avg_long.c or spec_avg.c code in the past.

Edited by Evan Goetz

Merge request reports

Loading