-
-
Notifications
You must be signed in to change notification settings - Fork 232
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MRG+1] Implement strategy from scikit-learn/scikit-learn#20506 #455
Conversation
I'm tracking this here so adding myself to the thread in case I can be useful. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much for fixing this so quickly. A+ code
Thank you. Thank you. |
This has not been pushed to conda-forge yet. Are there any blockers preventing the conda-forge update? |
Thanks for the link to the conda side :) |
Description
According to scikit-learn/scikit-learn#20506 and scikit-learn/scikit-learn#21144 (comment), it seems
if_delegate_has_method
is deprecated in favor ofavailable_if
. This is an attempt to switch toavailable_if
Fixes #454
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Checklist: