Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: afw
-
Labels:
-
Story Points:2
-
Epic Link:
-
Sprint:AP F18-6
-
Team:Alert Production
Description
While the changes introduced in DM-15682 passed formal testing, I discovered that they do not pass if afw is built against NumPy 1.13 – that version of numpy includes more whitespace in the array than the test expects:
AssertionError: '[[0. 0.]\n [0. 0.]]' not found in '[[ 0. 0.]\n [ 0. 0.]], bbox=(minimum=(0, 0), maximum=(1, 1))'
|
While the build failure can be worked around by upgrading NumPy to 1.14 or later (which will soon be required anyway), the consensus on #dm-build-problems was that any future formatting changes in NumPy would break the test again.
Please modify the tests so that they're less sensitive to NumPy's formatting decisions, preferably while keeping some diagnostic of whether the array is included in the string.
Attachments
Issue Links
- relates to
-
DM-15682 Add str() for afw::Image and afw::Mask
- Done