-
-
Notifications
You must be signed in to change notification settings - Fork 26.5k
ENH Add verbose to classifier regressor chains #23977
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
ENH Add verbose to classifier regressor chains #23977
Conversation
…_samples, n_classes
…e chain, per order parameter
…fiegel/scikit-learn into verbose-to-classifier-regressor-chains pulling from GitHub
Co-authored-by: Guillaume Lemaitre <[email protected]> Co-authored-by: Chiara Marmo <[email protected]>
thomasjpfan
left a comment
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!
thomasjpfan
left a comment
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.
Minor comment on the whats new. Otherwise LGTM
doc/whats_new/v1.2.rst
Outdated
| :mod:`sklearn.multioutput` | ||
| .......................... | ||
|
|
||
| - |API| Added boolean `verbose` flag to classes: |
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.
I'll say this is a feature:
| - |API| Added boolean `verbose` flag to classes: | |
| - |Feature| Added boolean `verbose` flag to classes: |
According to our legend:
scikit-learn/doc/whats_new/changelog_legend.inc
Lines 11 to 12 in 79c21c5
| - |API|: you will need to change your code to have the same effect in the | |
| future; or a feature will be removed in the future. |
"API" is used for API changes.
jeremiedbb
left a comment
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. Thanks @lucyleeow
Reference Issues/PRs
Fixes #16380
Closes #18947 (supersedes)
What does this implement/fix? Explain your changes.
Adds a boolean verbose option to
ClassifierChainandRegressorChain.Any other comments?