-
-
Notifications
You must be signed in to change notification settings - Fork 25.1k
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
MAINT Clean deprecation for 1.2: learning_rate and init in TSNE #24389
Conversation
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.
LGTM! Just a quickfix.
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.
LGTM
I'm not sure about codecov here. It complains that a branch with init="random" is not covered but there's at least 1 test that sets init="random". |
Yep, I don't understand codecov. There are several tests with |
Merging since we have 2 approvals and that I don't trust codecov after checking the tests. |
…it-learn#24389) Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>
Handle the ending deprecation for
learning_rate
andinit
by:versionchanged
filterwarnings
in the tests.