roman-corgi / corgidrp

Data Reduction Pipeline for the Roman Coronagraph Instrument
BSD 3-Clause "New" or "Revised" License
5 stars 4 forks source link

accommodation of EMGAIN_M, EMGAIN_A, and CMDGAIN in that order of priority #174

Closed kjl0025 closed 3 months ago

kjl0025 commented 3 months ago

Describe your changes

Where absolute EM gain is desired, this PR tries to retrieve the header key measured EM gain (EMGAIN_M), and if applied EM gain (EMGAIN_A) if that fails, and commanded EM gain (CMDGAIN) if that fails. For sorting by EM gain, commanded EM gain or applied EM gain is used instead of measured EM gain (since measured EM gain will vary frame to frame even if the commanded EM gain is the same for them all). (same changes as in deleted emgain_a branch but w/o whitespace differences)

Type of change

New feature (non-breaking change which adds functionality)

Reference any relevant issues (don't forget the #)

108

Checklist before requesting a review

kjl0025 commented 3 months ago

I had to turn off the VS Code option that enforced the deletion of trailing whitespaces.