You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While reviewing #1803 a somewhat related issue was found. See #1803 (comment) for the initial comment.
asdf._tests._helpers.assert_tree_match discards array masks during the comparison.
This is caused by 2 related issues (both covered here) and this line:
While reviewing #1803 a somewhat related issue was found. See #1803 (comment) for the initial comment.
asdf._tests._helpers.assert_tree_match discards array masks during the comparison.
This is caused by 2 related issues (both covered here) and this line:
asdf/asdf/_tests/_helpers.py
Line 55 in 33e7095
The call to
__array__
drops the mask:returns
So the mask never makes it to
assert_array_equal
. Furthermore:Does not fail (as it appears
assert_array_equal
(andassert_equal
) ignore masks.As our test suite contains several masked arrays it seems worthwhile to:
assert_tree_match
to handle the masks (or remove that test function)np.testing
that might handle a masked array (I'm not sure if there's a drop-in replacement or some argument we can provide).The text was updated successfully, but these errors were encountered: