Skip to content
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

replace deprecated grid_scores_ attribute #218

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Conversation

mdb571
Copy link

@mdb571 mdb571 commented Sep 10, 2022

grid_scores_ attribute is deprecated and will be removed in sklearn 1.2 .

This PR proposes to use the preferred cv_results_ in favour

@lizgehret
Copy link
Member

Thanks @mdb571! We are in the middle of some PR triage and review right now, and will take a look at this soon. In the meantime, can you fix these lint errors? Looks like a couple of line lengths are just a bit too long. Thanks!

@valentynbez
Copy link
Contributor

valentynbez commented Dec 30, 2022

Irrelevant for now, should be implemented during bumping scikit-learn version from 1.0 to 1.2.0.

@lizgehret
Copy link
Member

Thanks @valentynbez! We will either include skbio 1.2 in a patch release within the next couple of weeks, or we will hold off until our next release (~Feb 2023) to include it, pending any reverse dependency issues we may run into. We will make an announcement on the forum with details once we have things worked out on our end!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Analysis
Development

Successfully merging this pull request may close these issues.

None yet

4 participants