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

Afterburner for DP0.2 to populate ForcedSourceOnDiaObject fluxes in

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      Afterburner for DP0.2 to recover from DM-35322.

      Aborted plan is to use the fact that photocalibs are flat (1 parameter) per detector for imsim to write a very specialized cheap task that only works for this particular rescue case.

      Going forward with plan to make a set of data products that look exactly like they would if we did it right the first time.

      Need a good name for the "final" table: recoveredForcedSourceOnDiaObjectTable? finalForcedSourceOnDiaObjectTable?

        Attachments

          Activity

          Hide
          yusra Yusra AlSayyad added a comment -

          This is ready for review. I tested on the IDF with:

          pipetask run -b s3://butler-us-central1-panda-dev/dc2/butler-external.yaml -o u/yusra/DM-35338 -d "instrument='LSSTCam-imSim' AND  skymap='DC2' AND tract=3828 AND patch=5" -p $OBS_LSST_DIR/pipelines/imsim/DRP.yaml#step8 --extend-run --skip-existing -j 4 
          

          takes anywhere between 3s and 10s per detector. 3e6 detectors.

          what should we call these tables?

          Show
          yusra Yusra AlSayyad added a comment - This is ready for review. I tested on the IDF with: pipetask run -b s3://butler-us-central1-panda-dev/dc2/butler-external.yaml -o u/yusra/DM-35338 -d "instrument='LSSTCam-imSim' AND skymap='DC2' AND tract=3828 AND patch=5" -p $OBS_LSST_DIR/pipelines/imsim/DRP.yaml#step8 --extend-run --skip-existing -j 4 takes anywhere between 3s and 10s per detector. 3e6 detectors. what should we call these tables?
          Hide
          yusra Yusra AlSayyad added a comment -

          Convo on obs_lsst PR and what to name the tables: https://lsstc.slack.com/archives/C8EEUGDSA/p1657814390731469

          Show
          yusra Yusra AlSayyad added a comment - Convo on obs_lsst PR and what to name the tables: https://lsstc.slack.com/archives/C8EEUGDSA/p1657814390731469
          Hide
          yusra Yusra AlSayyad added a comment - - edited

          OK Colin, thanks for starting the review. I'm embarrassed that the PR wasn't ready the first time you looked. Per convo ^^^ I added a DRP-afterburner.yaml to obs_lsst with just the new task that writes out to the same name as the old one.

          I am worried that we'll end up accidentally mixing old and new files somehow, but I am cool with this being an experiment for the future as one possible way to fix a data product.

          Show
          yusra Yusra AlSayyad added a comment - - edited OK Colin, thanks for starting the review. I'm embarrassed that the PR wasn't ready the first time you looked. Per convo ^^^ I added a DRP-afterburner.yaml to obs_lsst with just the new task that writes out to the same name as the old one. I am worried that we'll end up accidentally mixing old and new files somehow, but I am cool with this being an experiment for the future as one possible way to fix a data product.
          Hide
          ctslater Colin Slater added a comment -

          Looks good, I was able to run it and see that mergedForcedSourceOnDiaObject has good values from all of the plugins.

          Show
          ctslater Colin Slater added a comment - Looks good, I was able to run it and see that mergedForcedSourceOnDiaObject has good values from all of the plugins.
          Hide
          yusra Yusra AlSayyad added a comment -

          Only merged to v23.0.x. Serves no purpose on main.

          Show
          yusra Yusra AlSayyad added a comment - Only merged to v23.0.x. Serves no purpose on main.

            People

            Assignee:
            yusra Yusra AlSayyad
            Reporter:
            yusra Yusra AlSayyad
            Reviewers:
            Colin Slater
            Watchers:
            Colin Slater, Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.