DOC Prefer ifskip and reference scikit-learn#21361

jjerphan · ogrisel · jjerphan · commit 5ca918a131cc · 2021-11-18T14:23:23.000+01:00
Co-authored-by: Olivier Grisel &lt;olivier.grisel@ensta.org&gt;
diff --git a/sklearn/utils/__init__.py b/sklearn/utils/__init__.py
@@ -42,6 +42,7 @@
     check_scalar,
 )
 from .. import get_config
+
 from ..utils.fixes import threadpool_info
 
 # Do not deprecate parallel_backend and register_parallel_backend as they are
diff --git a/sklearn/utils/_testing.py b/sklearn/utils/_testing.py
@@ -457,9 +457,9 @@ def set_random_state(estimator, random_state=0):
         not joblib.parallel.mp, reason="joblib is in serial mode"
     )
 
-    fails_if_unstable_openblas = pytest.mark.xfail(
+    fails_if_unstable_openblas = pytest.mark.skipif(
         _in_unstable_openblas_configuration(),
-        reason="OpenBLAS is unstable for this configuration",
+        reason="OpenBLAS is unstable for this configuration (#21361)",
     )
     #  Decorator for tests involving both BLAS calls and multiprocessing.
     #

Original file line number	Diff line number	Diff line change
`@@ -42,6 +42,7 @@`
`42`	`42`	`check_scalar,`
`43`	`43`	`)`
`44`	`44`	`from .. import get_config`
	`45`	`+`
`45`	`46`	`from ..utils.fixes import threadpool_info`
`46`	`47`
`47`	`48`	`# Do not deprecate parallel_backend and register_parallel_backend as they are`
Original file line number	Diff line number	Diff line change
`@@ -457,9 +457,9 @@ def set_random_state(estimator, random_state=0):`
`457`	`457`	`not joblib.parallel.mp, reason="joblib is in serial mode"`
`458`	`458`	`)`
`459`	`459`
`460`		`- fails_if_unstable_openblas = pytest.mark.xfail(`
	`460`	`+ fails_if_unstable_openblas = pytest.mark.skipif(`
`461`	`461`	`_in_unstable_openblas_configuration(),`
`462`		`- reason="OpenBLAS is unstable for this configuration",`
	`462`	`+ reason="OpenBLAS is unstable for this configuration (#21361)",`
`463`	`463`	`)`
`464`	`464`	`# Decorator for tests involving both BLAS calls and multiprocessing.`
`465`	`465`	`#`