Fix Version/s: None
Sprint:Alert Production S17 - 2
The constructor for lsst::afw::math::StatisticsControl has some arguments with default values, but the pybind11 wrapper omits them and only provides a default constructor. This breaks some existing code. I propose to wrap the constructor as written (which will be quite pleasant to use with named arguments) rather than fix the existing Python code that relies on being able to specify arguments to the constructor.
Pim Schellart [X] (Inactive) added a comment -
Looks good. Although I think I have an alternative fix for it on
Pim Schellart [X] (Inactive) added a comment - Looks good. Although I think I have an alternative fix for it on DM-9156 ;(
Trivial addition, but I also took the opportunity to clean up statistics.cc in a separate commit.