Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: afw
-
Labels:
-
Story Points:12.9
-
Epic Link:
-
Sprint:DRP F16-5, DRP F16-6, DRP S17-1
-
Team:Data Release Production
Description
The generated wrappers will live parallel to the Swig wrappers. This ticket only covers the C++ wrappers themselves, not the Python layer on top (which will continue to use the old wrappers) all work will stay on a separate branch and will not be merged to master until DM-6168 is complete.
The tests included in this ticket are:
- testFluxFromABMag.py
- testWcs835.py
- testHeader.py
- testTableArchiveImport.py
- testMaskedImagePersistence1.py
- testImageIo2.py
- testTicket2352.py
- testTicket2905.py
- testImagePersistence1.py
- testScaledPlus.py
- testRandom1.py
- testImageTestUtils.py
- testKernel.py
- testWcsFitsTable.py
- testTicket2162.py
- testStatisticsOverloads.py
- testMakeWcs.py
- testStatisticsMasked.py
- testKernelIo1.py
- testEllipse.py
- testRowColumnStats.py
- testMethods.py
- testKernelImagesForRegion.py
- statBug1697.py
Attachments
Issue Links
- blocks
-
- Done
-
- Done
-
- Done
- is blocked by
-
DM-8363 Add macro for Pybind11 wrapping of LSST_CONTROL_FIELD
- Done
-
- Done
- is cloned by
-
- Done
Activity
testSpatialCell.py was wrapped in DM-6298 with afw.math but the last test requires afw.image. The test is currently skipped but this test should be executed to verify that it passes before this ticket is completed.
Fred Moolekamp
added a comment - testSpatialCell.py was wrapped in DM-6298 with afw.math but the last test requires afw.image. The test is currently skipped but this test should be executed to verify that it passes before this ticket is completed. Moved pickling tests to DM-8415.
This removes:
- testPickles.py
- imagePickle.py
- testTicket2233.py
from this ticket.
Pim Schellart [X] (Inactive)
added a comment - Moved pickling tests to DM-8415 .
This removes:
testPickles.py
imagePickle.py
testTicket2233.py
from this ticket. Moved testDs9.py to DM-7799.
Moved testWarper.py to DM-7801.
Pim Schellart [X] (Inactive)
added a comment - Moved testWarper.py to DM-7801 . Moved the following tests to DM-8417.
- testValidPolygon
- testTableUtils
- testApCorrMap
Pim Schellart [X] (Inactive)
added a comment - Moved the following tests to DM-8417 .
testValidPolygon
testTableUtils
testApCorrMap
Note that all commits will be squashed into one wrap lsst::image commit before merge to epic branch. But kept separate for review.
Pim Schellart [X] (Inactive)
added a comment - Note that all commits will be squashed into one wrap lsst::image commit before merge to epic branch. But kept separate for review. Split off my work as DM-8447.
Krzysztof Findeisen
added a comment - Split off my work as DM-8447 . Everything looks good. I had a few minor comments and some questions about a slight change in the API in a few tests. I did't see any notes about Jenkins, so make sure that it passes py 2 and 3 after rebasing with DM-6168.
Fred Moolekamp
added a comment - Everything looks good. I had a few minor comments and some questions about a slight change in the API in a few tests. I did't see any notes about Jenkins, so make sure that it passes py 2 and 3 after rebasing with DM-6168 . Merged.
Pim Schellart [X] (Inactive)
added a comment - Merged. 
testChebyshevBoundedField requires lsst.afw.image but it was originally placed in
DM-6298since lsst.afw.image was not explicitly imported (even though afw.image is required to be wrapped for the test). The Description has been modified to reflect the change.