Replace sklearn.utils.metaestimators.if_delegate_has_method with available_if #693
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reference issues/PRs
Fixes #680.
Types of changes
Description
This replaces occurrences of the deprecated
sklearn.utils.metaestimators.if_delegate_has_method
, which was removed in scikit-learn version 1.3, withsklearn.utils.metaestimators.available_if
. Before this change, the pytest tests were throwing errors for me. This is a breaking change because the minimum version of scikit-learn has to be increased from 0.23.1 to 1.1.0.Screenshots (if appropriate)
Any other comments?
I am still not passing all of the tests, but I think the failing tests are related to other issues.
Checklist
flake8
to check my Python changes.pytest
to check this on Python tests.