8000 TST Fix failing tests catched by azure cron job (#23167) · thomasjpfan/scikit-learn@2b0d2aa · GitHub
[go: up one dir, main page]

8000
Skip to content

Commit 2b0d2aa

Browse files
authored
TST Fix failing tests catched by azure cron job (scikit-learn#23167)
1 parent 5223669 commit 2b0d2aa

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

sklearn/neighbors/tests/test_neighbors.py

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1535,7 +1535,9 @@ def test_neighbors_metrics(
15351535
and algorithm == "brute"
15361536
and sp_version >= parse_version("1.6.0")
15371537
):
1538-
with pytest.warns(FutureWarning):
1538+
with pytest.warns((FutureWarning, DeprecationWarning)):
1539+
# For float64 WMinkowskiDistance raises a FutureWarning,
1540+
# for float32 scipy raises a DeprecationWarning
15391541
results[algorithm] = neigh.kneighbors(X_test, return_distance=True)
15401542
else:
15411543
results[algorithm] = neigh.kneighbors(X_test, return_distance=True)
@@ -1578,12 +1580,9 @@ def test_kneighbors_brute_backend(
15781580
# wminkoski is deprecated in SciPy 1.6.0 and removed in 1.8.0
15791581
warn_context_manager = nullcontext()
15801582
if metric == "wminkowski" and sp_version >= parse_version("1.6.0"):
1581-
if global_dtype == np.float64:
1582-
# Warning from sklearn.metrics._dist_metrics.WMinkowskiDistance
1583-
warn_context_manager = pytest.warns(FutureWarning)
1584-
if global_dtype == np.float32:
1585-
# Warning from Scipy
1586-
warn_context_manager = pytest.warns(DeprecationWarning)
1583+
# For float64 WMinkowskiDistance raises a FutureWarning,
1584+
# for float32 scipy raises a DeprecationWarning
1585+
warn_context_manager = pytest.warns((FutureWarning, DeprecationWarning))
15871586

15881587
for metric_params in metric_params_list:
15891588
p = metric_params.pop("p", 2)
@@ -1597,6 +1596,7 @@ def test_kneighbors_brute_backend(
15971596
)
15981597

15991598
neigh.fit(X_train)
1599+
16001600
with warn_context_manager:
16011601
with config_context(enable_cython_pairwise_dist=False):
16021602
# Use the legacy backend for brute
@@ -2105,7 +2105,9 @@ def test_radius_neighbors_brute_backend(
21052105
# wminkoski is deprecated in SciPy 1.6.0 and removed in 1.8.0
21062106
warn_context_manager = nullcontext()
21072107
if metric == "wminkowski" and sp_version >= parse_version("1.6.0"):
2108-
warn_context_manager = pytest.warns(FutureWarning)
2108+
# For float64 WMinkowskiDistance raises a FutureWarning,
2109+
# for float32 scipy raises a DeprecationWarning
2110+
warn_context_manager = pytest.warns((FutureWarning, DeprecationWarning))
21092111

21102112
for metric_params in metric_params_list:
21112113
p = metric_params.pop("p", 2)

0 commit comments

Comments
 (0)
0