-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
DOC Update default values CalibratedClassifierCV docstring #17494
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
DOC Update default values CalibratedClassifierCV docstring #17494
Conversation
sklearn/preprocessing/_data.py
Outdated
@@ -1754,6 +1754,18 @@ def normalize(X, norm='l2', *, axis=1, copy=True, return_norm=False): | |||
An array of norms along given axis for X. | |||
When X is sparse, a NotImplementedError will be raised | |||
for norm 'l1' or 'l2'. | |||
|
|||
Warning:: Risk of data leak |
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.
Thanks for the PR! This is an unrelated change and shouldn't be in this PR, I think :)
Sorry, something went wrong.
All reactions
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.
Thanks, I updated it to contain only the relevant commit.
Sorry, something went wrong.
All reactions
Updated defalt values of parameters in CalibratedClassifierCV
115e4b3
to
fdae0d4
Compare
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 for the PR!
Sorry, something went wrong.
All reactions
Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
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 @asubramaniyan !
LGTM
Sorry, something went wrong.
All reactions
…arn#17494) Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
…arn#17494) Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
amueller
thomasjpfan
Successfully merging this pull request may close these issues.
Updated default values of parameters in CalibratedClassifierCV
Reference Issues/PRs
#15761
Addresses sklearn.calibration.CalibratedClassifierCV
cc NeilBotelho
#DataUmbrella