From 8404e916a2412a4ac3474f39534398802d5e753d Mon Sep 17 00:00:00 2001 From: Christian Veenhuis <124370897+ChVeen@users.noreply.github.com> Date: Thu, 17 Apr 2025 20:45:30 +0000 Subject: [PATCH] remove unused vars --- sklearn/utils/estimator_checks.py | 1 - sklearn/utils/tests/test_estimator_checks.py | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/sklearn/utils/estimator_checks.py b/sklearn/utils/estimator_checks.py index 6c3d16d98d7fb..d1c8d5d3fb610 100644 --- a/sklearn/utils/estimator_checks.py +++ b/sklearn/utils/estimator_checks.py @@ -889,7 +889,6 @@ def callback( # as xfail. check_result["status"] = "xfail" else: - failed = True check_result["status"] = "failed" if on_fail == "warn": diff --git a/sklearn/utils/tests/test_estimator_checks.py b/sklearn/utils/tests/test_estimator_checks.py index 4e573c8d1793f..c010a007d7525 100644 --- a/sklearn/utils/tests/test_estimator_checks.py +++ b/sklearn/utils/tests/test_estimator_checks.py @@ -1373,8 +1373,8 @@ def callback( expected_failed_checks = _get_expected_failed_checks(est) # This is to make sure we test a class that has some expected failures assert len(expected_failed_checks) > 0 - with warnings.catch_warnings(record=True) as records: - logs = check_estimator( + with warnings.catch_warnings(record=True): + check_estimator( est, expected_failed_checks=expected_failed_checks, on_fail=None,