Skip to content

Add frequency and SFT times to spectrogram output, improve file naming

Evan Goetz requested to merge evan-goetz/lalsuite:sgram-data into master

Description

lalpulsar_spec_avg spectrogram output was only the data in the spectrogram, and none of the associated frequencies or times. This merge request now adds that data content to the spectrogram output file.

In addition, the filenames did not have any file suffix, which this now adds.

Unit tests have been updated to ensure functionality.

Closes #606 (closed)

API Changes and Justification

Backwards Compatible Changes

  • This change does not modify any class/function/struct/type definitions in a public C header file or any Python class/function definitions
  • This change adds new classes/functions/structs/types to a public C header file or Python module

Backwards Incompatible Changes

  • This change modifies an existing class/function/struct/type definition in a public C header file or Python module
  • This change removes an existing class/function/struct/type from a public C header file or Python module

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

Suggest @ansel-neunzert should have a look at this to confirm it does what is needed and advertised since we need this update for Fscans. @karl-wette and @david-keitel may wish to have a look as well.

Edited by Evan Goetz

Merge request reports