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

Investigate cause of increase in Principal Color scatter in tract 9697

    XMLWordPrintable

    Details

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

      Description

      Yusra AlSayyad points out that there is an increase in the "wired" wPerp Principal Color scatter for tract 9697 between the w_2018_36 and w_2018_38 HSC RC2 processing runs (and it has persisted ever since). This is the blue "wired" curves: https://lsst-web.ncsa.illinois.edu/~hchiang2/RC2_w_2018_36/pipe_analysis/qaPlots/color/tract-9697/plot-t9697-griPSF-wFit-fit.png vs. https://lsst-web.ncsa.illinois.edu/~hchiang2/RC2_w_2018_38/pipe_analysis/qaPlots/color/tract-9697/plot-t9697-griPSF-wFit-fit.png
      Investigate the cause.

        Attachments

          Activity

          Hide
          lauren Lauren MacArthur added a comment - - edited

          I tracked down this issue. It turns out there were some inconsistencies in doing the transformations to the Principal Colors for the fit and wired coefficients (we do this for both PSF mags and CModel mags and they were getting intermixed). This is fixed here and we now also use the same points in all mean and stddev calculations (i.e. those that were used in the fit and survived the sigma cuts).

          This is the pre-fix plot:
          https://lsst-web.ncsa.illinois.edu/~hchiang2/RC2_w_2019_10/pipe_analysis/qaPlots/color/tract-9697/plot-t9697-griPSF-wFit-fit.png
          And here is the same post-fix:
          https://lsst-web.ncsa.illinois.edu/~lauren/lauren/DM-18532/plots/color/tract-9697/plot-t9697-griPSF-wFit-fit.png

          Updated plot for all tracts can be found (for now!) in: https://lsst-web.ncsa.illinois.edu/~lauren/lauren/DM-18532/plots/color/

          (Oh, and I should add that the issue revealed itself when I changed the flux-to-plot-against column from PSF to CModel. The default set in coaddAnalysis.py is also adopted in colorAnalysis.py...I may not have noticed otherwise!)

          Show
          lauren Lauren MacArthur added a comment - - edited I tracked down this issue. It turns out there were some inconsistencies in doing the transformations to the Principal Colors for the fit and wired coefficients (we do this for both PSF mags and CModel mags and they were getting intermixed). This is fixed here and we now also use the same points in all mean and stddev calculations (i.e. those that were used in the fit and survived the sigma cuts). This is the pre-fix plot: https://lsst-web.ncsa.illinois.edu/~hchiang2/RC2_w_2019_10/pipe_analysis/qaPlots/color/tract-9697/plot-t9697-griPSF-wFit-fit.png And here is the same post-fix: https://lsst-web.ncsa.illinois.edu/~lauren/lauren/DM-18532/plots/color/tract-9697/plot-t9697-griPSF-wFit-fit.png Updated plot for all tracts can be found (for now!) in: https://lsst-web.ncsa.illinois.edu/~lauren/lauren/DM-18532/plots/color/ (Oh, and I should add that the issue revealed itself when I changed the flux-to-plot-against column from PSF to CModel. The default set in coaddAnalysis.py is also adopted in colorAnalysis.py ...I may not have noticed otherwise!)
          Hide
          lauren Lauren MacArthur added a comment -

          Thanks again for catching this issue.  Can you give this a look?

          Show
          lauren Lauren MacArthur added a comment - Thanks again for catching this issue.  Can you give this a look?
          Hide
          lauren Lauren MacArthur added a comment -

          Thanks for the review! Merged to master.

          Show
          lauren Lauren MacArthur added a comment - Thanks for the review! Merged to master.

            People

            Assignee:
            lauren Lauren MacArthur
            Reporter:
            lauren Lauren MacArthur
            Reviewers:
            Yusra AlSayyad
            Watchers:
            Lauren MacArthur, Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                CI Builds

                No builds found.