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

Add Jacobian BoundedField accessor to SkyWcs

    Details

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

      Description

      We've agreed that it should not be the responsibility of the PhotoCalib to include a Jacobian derived from the WCS, because we don't always want to apply that.  Instead, SkyWcs should have the ability (given a pixel bounding box) to return a BoundedField that evaluates to the determinant of the Jacobian of the pixelToSky transform (aka the pixel area in angular units) at every point in that bounding box efficiently.  This probably involves a new BoundedField class (though it's possible there's a way to do this with TransformBoundedField).

      A reference implementation can be found in meas_mosaic's calculateJacobian functions, though a different approach to vectorization may be necessary to get good performance from AST.

        Attachments

          Issue Links

            Activity

            jbosch Jim Bosch created issue -
            jbosch Jim Bosch made changes -
            Field Original Value New Value
            Risk Score 0
            jbosch Jim Bosch made changes -
            Link This issue relates to DM-14020 [ DM-14020 ]
            jbosch Jim Bosch made changes -
            Link This issue blocks DM-15254 [ DM-15254 ]
            Parejkoj John Parejko made changes -
            Watchers Jim Bosch, John Parejko [ Jim Bosch, John Parejko ] Jim Bosch, John Parejko, Russell Owen [ Jim Bosch, John Parejko, Russell Owen ]
            Parejkoj John Parejko made changes -
            Watchers Jim Bosch, John Parejko, Russell Owen [ Jim Bosch, John Parejko, Russell Owen ] Jim Bosch, John Parejko, John Swinbank, Russell Owen [ Jim Bosch, John Parejko, John Swinbank, Russell Owen ]
            swinbank John Swinbank made changes -
            Epic Link DM-11783 [ 34260 ]
            swinbank John Swinbank made changes -
            Assignee John Parejko [ parejkoj ]
            swinbank John Swinbank made changes -
            Sprint AP F18-4 [ 749 ]
            swinbank John Swinbank made changes -
            Sprint AP F18-4 [ 749 ] AP F18-4, AP F18-5 [ 749, 750 ]
            Parejkoj John Parejko made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            Parejkoj John Parejko made changes -
            Attachment wcsJacobian_muddling.py [ 34402 ]
            Parejkoj John Parejko made changes -
            Reviewers Paul Price [ price ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            swinbank John Swinbank made changes -
            Sprint AP F18-4, AP F18-5 [ 749, 750 ] AP F18-4, AP F18-5, AP F18-6 [ 749, 750, 751 ]
            price Paul Price made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Parejkoj John Parejko made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-16466 [ DM-16466 ]

              People

              • Assignee:
                Parejkoj John Parejko
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Paul Price
                Watchers:
                Eli Rykoff, Jim Bosch, John Parejko, John Swinbank, Paul Price, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel