Skip to content

Commit 48babd0

Browse files
Remove test_cond_nan_input from TestCond (#2221)
This PR suggests removing `test_cond_input_nan` due to undefined behavior for different backend in different versions of OpenBLAS and OneMKL to avoid possible future falls _Checking all input for nan's is expensive, so MKL lapack functions don't do it_
1 parent 11487c5 commit 48babd0

File tree

1 file changed

+0
-10
lines changed

1 file changed

+0
-10
lines changed

dpnp/tests/test_linalg.py

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -341,16 +341,6 @@ def test_bool(self, p):
341341
expected = numpy.linalg.cond(a, p=p)
342342
assert_dtype_allclose(result, expected)
343343

344-
@pytest.mark.parametrize("p", [-dpnp.inf, -1, 1, dpnp.inf, "fro"])
345-
def test_nan_input(self, p):
346-
a = generate_random_numpy_array((3, 3))
347-
a[1, 1] = numpy.nan
348-
ia = dpnp.array(a)
349-
350-
result = dpnp.linalg.cond(ia, p=p)
351-
expected = numpy.linalg.cond(a, p=p)
352-
assert_dtype_allclose(result, expected)
353-
354344
@pytest.mark.parametrize(
355345
"p", [None, -dpnp.inf, -2, -1, 1, 2, dpnp.inf, "fro"]
356346
)

0 commit comments

Comments
 (0)