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

Add TE1 and TE2 KPMs to validate_drp using code developed in DM-3040

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Validation
    • Labels:
      None

      Description

      In Summer 2015, Bob Armstrong wrote code to calculate the residual ellipticity KPM: TE1 and TE2

      https://jira.lsstcorp.org/browse/DM-3040

      Add this code to add TE1 and TE2 to the KPMs calculated in validate_drp

        Attachments

          Issue Links

          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9961 Add treecorr to validate_drp and lsst_ci dependencies

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9960 Create wrapper install packages for libffi and cffi

          • Won't Fix
          is duplicated by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7004 Add ellipticity measurement to validate_drp

          • Invalid
          relates to

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-296 Add treecorr and libffi to lsst_ci dependencies

          • Implemented

          Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. DM-3040 KPM Measurement: Residual PSF Ellipticity Correlations (TE1), FY15

          • Done

            Activity

            Ascending order - Click to sort in descending order
            9 older comments
            Hide
            Permalink
            wmwood-vasey Michael Wood-Vasey added a comment -

            Thanks Tim Jenness for the link to settled practice for envPrepend.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Thanks Tim Jenness for the link to settled practice for envPrepend .
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            See comments in the PR.

            Show
            Parejkoj John Parejko added a comment - See comments in the PR.
            Hide
            Permalink
            wmwood-vasey Michael Wood-Vasey added a comment -

            John Parejko Could you take another look at this PR? Specifically, take a look at the tests I've added in tests/test_tex.py and double-check that I didn't write anything stupid in tests/test_util.py, which currently just checks the ellipticity calculation.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - John Parejko Could you take another look at this PR? Specifically, take a look at the tests I've added in tests/test_tex.py and double-check that I didn't write anything stupid in tests/test_util.py , which currently just checks the ellipticity calculation.
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            I've added some comments on the new code. There are still some open questions from before as well. Thanks for expanding the tests!

            Show
            Parejkoj John Parejko added a comment - I've added some comments on the new code. There are still some open questions from before as well. Thanks for expanding the tests!
            Hide
            Permalink
            wmwood-vasey Michael Wood-Vasey added a comment -

            Thanks, John Parejko, for all of the review work.

            Merged to master.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Thanks, John Parejko , for all of the review work. Merged to master.

              People

              Assignee:
              wmwood-vasey Michael Wood-Vasey
              Reporter:
              wmwood-vasey Michael Wood-Vasey
              Reviewers:
              John Parejko
              Watchers:
              John Parejko, Michael Wood-Vasey, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.