Skip to content

Conversation

glevv
Copy link
Contributor

@glevv glevv commented Dec 29, 2022

Reference Issues/PRs

Fixes #25066

What does this implement/fix? Explain your changes.

Deprecation of sample_weight parameter in predict method for KMeans

@glevv glevv changed the title Kmeans predict deprecation FIX deprecate sample_weight in predict method of KMeans Dec 29, 2022
@glevv glevv changed the title FIX deprecate sample_weight in predict method of KMeans FIX deprecate sample_weight in predict method of BaseKMeans Dec 30, 2022
Copy link
Member

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @glevv. Looks good. I think you can switch this PR to ready.

Copy link
Member

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jeremiedbb jeremiedbb added the Quick Review For PRs that are quick to review label Dec 30, 2022
@jeremiedbb jeremiedbb marked this pull request as ready for review December 30, 2022 15:06
Copy link
Member

@jjerphan jjerphan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you, @glevv.

Co-authored-by: Julien Jerphanion <git@jjerphan.xyz>
@jjerphan jjerphan enabled auto-merge (squash) January 2, 2023 13:42
@jjerphan jjerphan merged commit bd55522 into scikit-learn:main Jan 2, 2023
@glevv glevv deleted the kmeans-predict-deprecation branch January 2, 2023 16:00
jjerphan added a commit to jjerphan/scikit-learn that referenced this pull request Jan 3, 2023
jpangas pushed a commit to jpangas/scikit-learn that referenced this pull request Jan 4, 2023
jjerphan added a commit to jjerphan/scikit-learn that referenced this pull request Jan 20, 2023
jjerphan added a commit to jjerphan/scikit-learn that referenced this pull request Jan 20, 2023
@jjerphan jjerphan added this to the 1.3 milestone Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module:cluster Quick Review For PRs that are quick to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

KMeans.predict wrongly exposes a sample_weight argument
3 participants