Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-22797

Various FITS header fixes

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw, pipe_tasks
    • Labels:
    • Team:
      External

      Description

      The HSC team requests some changes to the output FITS headers:

      • FLUXMAG0 missing from the calexp header
      • Please do not erase original FITS header keywords in raw data for calexp (and deepCoadd_calexp). For example, FRAME_ID, EXP_ID, DATA-TYP, DET-ID, OBJECT, FILTER01, AZIMUTH, ALTITUDE, PROP-ID, DATE-OBS, MJD, etc.
      • In the deepCoadd_*Warp, DATE-OBS, MJD values are wrong.
      • Please do not omit the FITS keyword CD1_1, CD2_2, CD1_2, CD2_1 even when its value = 0.

        Attachments

          Issue Links

            Activity

            Hide
            price Paul Price added a comment -

            With the proposed changes, CalibrateTask will write a MAGZERO header keyword with its CCD zeropoint, in addition to the MAGZERO_RMS and MAGZERO_NOBJ header keywords it's already writing.

            Show
            price Paul Price added a comment - With the proposed changes, CalibrateTask will write a MAGZERO header keyword with its CCD zeropoint, in addition to the MAGZERO_RMS and MAGZERO_NOBJ header keywords it's already writing.
            Hide
            price Paul Price added a comment -

            Yusra AlSayyad, would you please review these changes (afw and pipe_tasks)?

            Show
            price Paul Price added a comment - Yusra AlSayyad , would you please review these changes ( afw and pipe_tasks )?
            Hide
            yusra Yusra AlSayyad added a comment -

            Looks good

            Show
            yusra Yusra AlSayyad added a comment - Looks good
            Hide
            price Paul Price added a comment -

            Thanks Yusra.

            Merged to master.

            Show
            price Paul Price added a comment - Thanks Yusra. Merged to master.
            Hide
            price Paul Price added a comment -
            Show
            price Paul Price added a comment - ( Jenkins is green .)

              People

              • Assignee:
                price Paul Price
                Reporter:
                price Paul Price
                Reviewers:
                Yusra AlSayyad
                Watchers:
                Hisanori Furusawa, John Parejko, Masayuki Tanaka, Paul Price, Robert Lupton, Tim Jenness, Yusra AlSayyad
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel