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

Add PSF mag plots to comparison scripts

    XMLWordPrintable

    Details

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

      Description

      These are very useful for QA in comparing measurements on two different processing runs of the same data/field. They got missed from the default list in the comparison scripts as they were inherited from the coaddAnalysis.py script whose Mag comparisons are all done w.r.t. the PSF mag, so would've been redundant.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          lauren Lauren MacArthur added a comment -

          Yusra AlSayyad, can you confirm this adds the plots you requested (thanks for noticing they were missing!). I also added a "highlight" to the sources that were used in the PSF modeling (calib_psfUsed = True). I have a sinking suspicion that may turn out to be more mess than it's worth in a real example, but my test datasets don't meet that criterion, so your opinion when looking at real-world use cases would be greatly appreciated.

          Show
          lauren Lauren MacArthur added a comment - Yusra AlSayyad , can you confirm this adds the plots you requested (thanks for noticing they were missing!). I also added a "highlight" to the sources that were used in the PSF modeling ( calib_psfUsed = True ). I have a sinking suspicion that may turn out to be more mess than it's worth in a real example, but my test datasets don't meet that criterion, so your opinion when looking at real-world use cases would be greatly appreciated.
          Hide
          Permalink
          yusra Yusra AlSayyad added a comment -

          Looked at the plots Sophie Reed made with this branch. Looks good!

          Show
          yusra Yusra AlSayyad added a comment - Looked at the plots Sophie Reed made with this branch. Looks good!
          Hide
          Permalink
          lauren Lauren MacArthur added a comment -

          Thanks Yusra! Merged to master.

          Show
          lauren Lauren MacArthur added a comment - Thanks Yusra! Merged to master.

            People

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

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.