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

Update pipe_tasks to allow for changing external calibrations

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: fgcmcal
    • Labels:
      None
    • Story Points:
      3
    • Epic Link:
    • Sprint:
      DRP F19-5, DRP F19-6 (Nov), DRP S20-2 (Jan)
    • Team:
      Data Release Production

      Description

      Update pipe_tasks and related configs to allow coadditions with different external PhotoCalib and SkyWcs calibrations, supporting jointcal, fgcmcal, and fgcmcal_tract for photometric calibrations, and jointcal for astrometric calibrations. Terminology is now matched to DM-21950.

      Originally: Following on DM-20161, with fgcmcal tract-based calibrations, run RC2 coadds.

        Attachments

          Issue Links

            Activity

            Hide
            erykoff Eli Rykoff added a comment -

            Coadds and multiband have been run, although there are some issues with the rerun chaining that is causing problems with some verify steps. I will have to investigate what is going on here and rerun. Early plots from `colorAnalysis` are on DM-21309.

            Show
            erykoff Eli Rykoff added a comment - Coadds and multiband have been run, although there are some issues with the rerun chaining that is causing problems with some verify steps. I will have to investigate what is going on here and rerun. Early plots from `colorAnalysis` are on DM-21309 .
            Hide
            erykoff Eli Rykoff added a comment -

            When the reruns are figured out, new tickets will need to be filed to add the ability to configure the calibration input.

            Show
            erykoff Eli Rykoff added a comment - When the reruns are figured out, new tickets will need to be filed to add the ability to configure the calibration input.
            Hide
            erykoff Eli Rykoff added a comment -

            Merging the changes on this ticket requires DM-20074 to be complete which (among other things) removes the `makeCoaddTempExp.py` override from `obs_subaru`.

            Show
            erykoff Eli Rykoff added a comment - Merging the changes on this ticket requires DM-20074 to be complete which (among other things) removes the `makeCoaddTempExp.py` override from `obs_subaru`.
            Hide
            erykoff Eli Rykoff added a comment -

            Removed blocker from DM-20074, as this ticket now, for the time being, updates `SubaruMakeCoaddTempExp` as well.

            Show
            erykoff Eli Rykoff added a comment - Removed blocker from DM-20074 , as this ticket now, for the time being, updates `SubaruMakeCoaddTempExp` as well.
            Hide
            mrawls Meredith Rawls added a comment -

            This ticket makes me very happy, because I rather disliked the term "ubercal" and things are much clearer now - thank you!

            I am marking this as "Reviewed," though there are a few small things I would like to see changed, which I left as GitHub comments. I trust you to make these small updates where appropriate, and then we can dust off our "uber" and forge ahead into the brave new "external" which breaks apart key functionalities and makes 1000% more sense!

            Show
            mrawls Meredith Rawls added a comment - This ticket makes me very happy, because I rather disliked the term "ubercal" and things are much clearer now - thank you! I am marking this as "Reviewed," though there are a few small things I would like to see changed, which I left as GitHub comments. I trust you to make these small updates where appropriate, and then we can dust off our "uber" and forge ahead into the brave new "external" which breaks apart key functionalities and makes 1000% more sense!

              People

              • Assignee:
                erykoff Eli Rykoff
                Reporter:
                erykoff Eli Rykoff
                Reviewers:
                Meredith Rawls
                Watchers:
                Eli Rykoff, Meredith Rawls
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel