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

Generate amp offset corrected sky frames

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: cp_pipe, pipe_drivers
    • Labels:
      None
    • Story Points:
      18
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      This ticket aims to regenerate HSC sky frames with the amp offset correction task turned on.

      HSC data are known to exhibit small (~few ADU) amp-to-amp offsets which persist post-ISR correction. The origin of this offset is believed to be a feature of the hardware, requiring an empirical correction (ampOffsetTask) after the fact. This task was implemented in DM-24389.

      HSC data undergo a full-focal-plane sky correction prior to coadd-level data processing, producing superior sky solutions on larger scales. This task was initially implemented in DM-9147, and converted to gen3 in DM-22534 (cpSkyTask). Following DM-33629, HSC sky frames were updated using the latest cpPipe implementation, however, amp-offset corrections were left switched off to allow for a direct comparison with the older data processing outputs.

      To that end, the calibration sky frames currently in use do not have amp offset corrections applied, therefore requiring regeneration on this ticket. Once tested as part of an RC2 run, these new amp-offset corrected sky frames should replace those currently in use for standard HSC data processing.

        Attachments

        1. _deepCoadd_9615_HSC-G_w08w11.mp4
          36.73 MB
        2. _deepCoadd_9615_HSC-R_w08w11.mp4
          37.99 MB
        3. _deepCoadd_calexp_9615_HSC-G_w08w11.mp4
          38.55 MB
        4. _deepCoadd_calexp_9615_HSC-R_w08w11.mp4
          40.29 MB
        5. gen3-corrected-sky-HSC-G.mp4
          1.42 MB
        6. gen3-corrected-sky-HSC-I.mp4
          281 kB
        7. gen3-corrected-sky-HSC-I2.mp4
          562 kB
        8. gen3-corrected-sky-HSC-R.mp4
          547 kB
        9. gen3-corrected-sky-HSC-R2.mp4
          464 kB
        10. gen3-corrected-sky-HSC-Y.mp4
          1.00 MB
        11. gen3-corrected-sky-HSC-Z.mp4
          1.05 MB
        12. vanilla-vs-corrected-sky-HSC-G.mp4
          2.71 MB
        13. vanilla-vs-corrected-sky-HSC-I.mp4
          509 kB
        14. vanilla-vs-corrected-sky-HSC-I2.mp4
          934 kB
        15. vanilla-vs-corrected-sky-HSC-R.mp4
          922 kB
        16. vanilla-vs-corrected-sky-HSC-R2.mp4
          904 kB
        17. vanilla-vs-corrected-sky-HSC-Y.mp4
          1.62 MB
        18. vanilla-vs-corrected-sky-HSC-Z.mp4
          1.84 MB

          Issue Links

            Activity

            Hide
            lskelvin Lee Kelvin added a comment - - edited

            In order to switch amp offset correction on by default, two things need to happen:

            1. merge the HSC config override in obs_subaru on this ticket.
            2. update the HSC/calib chained collection, which is referenced within HSC/defaults.

            To update HSC/calib, the old vanilla sky frame calibration collection (HSC/calib/DM-33629) needs to be removed, and the new sky frame calibration collection (HSC/calib/DM-32378) added. This can be achieved on the command line using the following:

            butler collection-chain /repo/main --mode remove HSC/calib HSC/calib/DM-33629
            

            followed by

            butler collection-chain /repo/main --mode prepend HSC/calib HSC/calib/DM-32378
            

            Ideally, merging this ticket to the main branch and performing the above chained collection redefinition should occur as close to one-another in time as possible.

            Show
            lskelvin Lee Kelvin added a comment - - edited In order to switch amp offset correction on by default, two things need to happen: merge the HSC config override in obs_subaru on this ticket. update the HSC/calib chained collection, which is referenced within HSC/defaults . To update HSC/calib , the old vanilla sky frame calibration collection ( HSC/calib/ DM-33629 ) needs to be removed, and the new sky frame calibration collection ( HSC/calib/ DM-32378 ) added. This can be achieved on the command line using the following: butler collection-chain /repo/main --mode remove HSC/calib HSC/calib/DM-33629 followed by butler collection-chain /repo/main --mode prepend HSC/calib HSC/calib/DM-32378 Ideally, merging this ticket to the main branch and performing the above chained collection redefinition should occur as close to one-another in time as possible.
            Hide
            lskelvin Lee Kelvin added a comment - - edited
            Show
            lskelvin Lee Kelvin added a comment - - edited Jenkins run for the PR associated with this ticket .
            Hide
            yusra Yusra AlSayyad added a comment -

            Looks good. OK to merge.

            • You actually have until the new shared stack gets built tomorrow to switch the default HSC/calib (if you want more sleep )
            • We need to advertise that if someone is processing RC2 with an older stack people need to use the SkyFrames in `HSC/calib/DM-33629`. I THINK doing {{ -i HSC/calib/DM-33629,HSC/RC2/defaults}} would do it, but Jim Bosch or Nate Lust name can correct me.

            We can continue to do QA after it is switched on.
            1) The images that I've been dying to see are the full visits with:

            • New Vanilla SkyFrames + skyCorr subtracted off vs. doAmpOffset = True + DM-32378 Sky Frames + skyCorr subtracted off.
              2) Eli Rykoff promised to make some a before and after photometric residual plots (summed over detector rows to maybe yield enough signal with just RC2 – e.g. 1D ramps vs 2D ramps)
              3) I promised to look at the warp diffs to see if they look more suitable for background matching.
            Show
            yusra Yusra AlSayyad added a comment - Looks good. OK to merge. You actually have until the new shared stack gets built tomorrow to switch the default HSC/calib (if you want more sleep ) We need to advertise that if someone is processing RC2 with an older stack people need to use the SkyFrames in `HSC/calib/ DM-33629 `. I THINK doing {{ -i HSC/calib/ DM-33629 ,HSC/RC2/defaults}} would do it, but Jim Bosch or Nate Lust name can correct me. We can continue to do QA after it is switched on. 1) The images that I've been dying to see are the full visits with: New Vanilla SkyFrames + skyCorr subtracted off vs. doAmpOffset = True + DM-32378 Sky Frames + skyCorr subtracted off. 2) Eli Rykoff promised to make some a before and after photometric residual plots (summed over detector rows to maybe yield enough signal with just RC2 – e.g. 1D ramps vs 2D ramps) 3) I promised to look at the warp diffs to see if they look more suitable for background matching.
            Hide
            lskelvin Lee Kelvin added a comment -

            That's great, thanks Yusra. Ticket merged just now, branch deleted, and the HSC/calib collection updated!

            Will add posts advertising the change on Slack into #dm-hsc-reprocessing and #dm-science-pipelines. Will also look at generating full visit images this week, and attach them to this ticket.

            Thanks again!

            Show
            lskelvin Lee Kelvin added a comment - That's great, thanks Yusra. Ticket merged just now, branch deleted, and the HSC/calib collection updated! Will add posts advertising the change on Slack into #dm-hsc-reprocessing and #dm-science-pipelines. Will also look at generating full visit images this week, and attach them to this ticket. Thanks again!
            Hide
            erykoff Eli Rykoff added a comment -

            I did promise this! I will look as soon as the upcoming RC2 processing gets far enough.

            Show
            erykoff Eli Rykoff added a comment - I did promise this! I will look as soon as the upcoming RC2 processing gets far enough.

              People

              Assignee:
              lskelvin Lee Kelvin
              Reporter:
              lskelvin Lee Kelvin
              Reviewers:
              Yusra AlSayyad
              Watchers:
              Eli Rykoff, Lee Kelvin, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.