Movatterモバイル変換

Skip to content

scikit-learn-contrib/imbalanced-learnPublic

NotificationsYou must be signed in to change notification settings
Fork1.3k
Star7k

API deprecate estimator_ in favor of estimators_ in CNN and OSS#1011

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

glemaitre merged 2 commits intoscikit-learn-contrib:masterfromglemaitre:is/908

Jul 10, 2023

Merged

API deprecate estimator_ in favor of estimators_ in CNN and OSS#1011

glemaitre merged 2 commits intoscikit-learn-contrib:masterfromglemaitre:is/908

Jul 10, 2023

Conversation

glemaitre

Copy link

Member

glemaitre commentedJul 9, 2023•
edited
Loading

closes#908

We should make sure that the documentation mentions the one-vs.-other strategy (wrong for CNN). Then it makes sense to store all pairs of k-NN instead of the last one from the loop.
We can then deprecateestimator_ and create a newestimators_ attribute.

@glemaitre

API deprecate estimator_ in favor of estimators_

b3621e7

@glemaitre

glemaitre changed the title~~API deprecate estimator_ in favor of estimators_~~API deprecate estimator_ in favor of estimators_ in CNN and OSS

@glemaitre

glemaitre marked this pull request as draft

July 9, 2023 22:06

@glemaitre

iter

bc26060

@glemaitre

glemaitre marked this pull request as ready for review

July 9, 2023 22:24

@glemaitre

glemaitre merged commit95e21e1 intoscikit-learn-contrib:master

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Labels

None yet

1 participant

@glemaitre

[8]ページ先頭

©2009-2025 Movatter.jp