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

Add calibUsed-only qa plots for astrometry and photometry

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_analysis
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • Sprint:
      DRP F17-2, DRP F17-3, DRP F17-4
    • Team:
      Data Release Production

      Description

      For qa analysis it is useful to have plots and statistics based solely on the objects that were used for any particular calibration. We have so far been making calib_psfUsed-only plots as the PSF modeling was the only provided such a flag in the output catalogs. As of DM-9050, we now also have calib_astrometryUsed & calib_photometryUsed flags for the astrometry and photometry fitting. This issue is to add the relevant/useful plots to the analysis scripts based on these objects.

        Attachments

          Activity

          Hide
          lauren Lauren MacArthur added a comment -

          Tim, would you mind doing a triplicate review? I have done the work on this ticket, DM-11542, and DM-11544. Each branch is based off the previous. You can look at the resulting plots for visit and coadds on lsst-dev at /datasets/hsc/repo/rerun/private/lauren/w_2017_34/plots/HSC-I/tract-9813.

          Show
          lauren Lauren MacArthur added a comment - Tim, would you mind doing a triplicate review? I have done the work on this ticket, DM-11542 , and DM-11544 . Each branch is based off the previous. You can look at the resulting plots for visit and coadds on lsst-dev at /datasets/hsc/repo/rerun/private/lauren/w_2017_34/plots/HSC-I/tract-9813 .
          Hide
          tmorton Tim Morton [X] (Inactive) added a comment -

          Hey Lauren-- Are there supposed to be plots with the calib_astrometryUsed and calib_photometryUsed flags? I only see calib_psfUsed in the directory you point to here; am I missing something?

          [tmorton@lsst-dev01 tract-9813]$ pwd
          /scratch/lauren/rerun/w_2017_34/plots/HSC-I/tract-9813
          [tmorton@lsst-dev01 tract-9813]$ ls *Used*.png
          plot-t9813-HSC-I-footNpix_calib_psfUsed-psfMagHist.png  plot-t9813-HSC-I-matches_mag_calib_psfUsed-psfMagHist.png
          plot-t9813-HSC-I-footNpix_calib_psfUsed-sky-stars.png   plot-t9813-HSC-I-matches_mag_calib_psfUsed-sky-stars.png
          

          Show
          tmorton Tim Morton [X] (Inactive) added a comment - Hey Lauren-- Are there supposed to be plots with the calib_astrometryUsed and calib_photometryUsed flags? I only see calib_psfUsed in the directory you point to here; am I missing something? [tmorton@lsst-dev01 tract-9813]$ pwd /scratch/lauren/rerun/w_2017_34/plots/HSC-I/tract-9813 [tmorton@lsst-dev01 tract-9813]$ ls *Used*.png plot-t9813-HSC-I-footNpix_calib_psfUsed-psfMagHist.png plot-t9813-HSC-I-matches_mag_calib_psfUsed-psfMagHist.png plot-t9813-HSC-I-footNpix_calib_psfUsed-sky-stars.png plot-t9813-HSC-I-matches_mag_calib_psfUsed-sky-stars.png
          Hide
          tmorton Tim Morton [X] (Inactive) added a comment -

          OK, maybe these were just made for the visit plots I see in the visit-1228 subdirectory? I see them there. Were they not intentionally not made for the coadd?

          Show
          tmorton Tim Morton [X] (Inactive) added a comment - OK, maybe these were just made for the visit plots I see in the visit-1228 subdirectory? I see them there. Were they not intentionally not made for the coadd?
          Hide
          lauren Lauren MacArthur added a comment -

          Those flags are not yet being propagated to the coadds, see DM-11866.

          Show
          lauren Lauren MacArthur added a comment - Those flags are not yet being propagated to the coadds, see DM-11866 .
          Hide
          tmorton Tim Morton [X] (Inactive) added a comment -

          Aha. Ok, that makes sense--- so at present these figures are only possible to do for visit plots?

          Show
          tmorton Tim Morton [X] (Inactive) added a comment - Aha. Ok, that makes sense--- so at present these figures are only possible to do for visit plots?
          Hide
          tmorton Tim Morton [X] (Inactive) added a comment -

          Looks good to me.

          Show
          tmorton Tim Morton [X] (Inactive) added a comment - Looks good to me.
          Hide
          lauren Lauren MacArthur added a comment -

          Thanks Tim. I created DM-11974 to make sure the coadd plots get added once DM-11866 lands.

          Show
          lauren Lauren MacArthur added a comment - Thanks Tim. I created DM-11974 to make sure the coadd plots get added once DM-11866 lands.

            People

            Assignee:
            lauren Lauren MacArthur
            Reporter:
            lauren Lauren MacArthur
            Reviewers:
            Tim Morton [X] (Inactive)
            Watchers:
            John Swinbank, Lauren MacArthur, Tim Morton [X] (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.