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

Update meas_mosaic to new transform-based API for afw::cameraGeom

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_mosaic
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      DRP F17-4
    • Team:
      Data Release Production

      Description

      With the merge of DM-5922, meas_mosaic is currently broken with errors like:

      src/mosaicfit.cc:2500:38: error: 'lsst::afw::cameraGeom::CameraTransformMap' has not been declared
                           afw::cameraGeom::CameraTransformMap::Transforms newTr;
                                            ^
      

      Please update meas_mosaic to the new afw::cameraGeom API.

        Attachments

          Issue Links

            Activity

            Hide
            lauren Lauren MacArthur added a comment - - edited

            Would you mind giving this a look? I can confirm identical meas_mosaic results for w_2017_34 + meas_mosaic:master and w_2017_35 + meas_mosaic:tickets/DM-11800

            An lsst_distrib + meas_mosaic:tickets/DM-11800 Jenkins is running (see also)

            Show
            lauren Lauren MacArthur added a comment - - edited Would you mind giving this a look? I can confirm identical meas_mosaic results for w_2017_34 + meas_mosaic:master and w_2017_35 + meas_mosaic:tickets/ DM-11800 An lsst_distrib + meas_mosaic:tickets/ DM-11800 Jenkins is running ( see also )
            Hide
            rowen Russell Owen added a comment -

            This looks perfect. Thank you for testing the new code against the old (important because meas_mosaic has few unit tests).

            Show
            rowen Russell Owen added a comment - This looks perfect. Thank you for testing the new code against the old (important because meas_mosaic has few unit tests).
            Hide
            lauren Lauren MacArthur added a comment -

            Thanks Russell. Merged to master.

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

              People

              • Assignee:
                lauren Lauren MacArthur
                Reporter:
                lauren Lauren MacArthur
                Reviewers:
                Russell Owen
                Watchers:
                Hsin-Fang Chiang, John Swinbank, Krzysztof Findeisen, Lauren MacArthur, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel