Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@lucyleeow
Copy link
Member

@lucyleeow lucyleeow commented Jul 22, 2022

Reference Issues/PRs

Fixes #16380
Closes #18947 (supersedes)

What does this implement/fix? Explain your changes.

Adds a boolean verbose option to ClassifierChain and RegressorChain.

Any other comments?

efiegel and others added 23 commits November 30, 2020 19:36
…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]>
Copy link
Member

@thomasjpfan thomasjpfan left a 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!

Copy link
Member

@thomasjpfan thomasjpfan left a 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

:mod:`sklearn.multioutput`
..........................

- |API| Added boolean `verbose` flag to classes:
Copy link
Member

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:

Suggested change
- |API| Added boolean `verbose` flag to classes:
- |Feature| Added boolean `verbose` flag to classes:

According to our legend:

- |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.

@thomasjpfan thomasjpfan added Quick Review For PRs that are quick to review module:multioutput labels Jul 26, 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.

LGTM. Thanks @lucyleeow

@jeremiedbb jeremiedbb merged commit be00837 into scikit-learn:main Jul 28, 2022
@lucyleeow lucyleeow deleted the verbose-to-classifier-regressor-chains branch July 29, 2022 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:multioutput Quick Review For PRs that are quick to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Verbose Option for Classifier and Regressor Chains

5 participants