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
[MGR] TransformedTargetRegressor passes fit_params to regressor #14890
[MGR] TransformedTargetRegressor passes fit_params to regressor #14890
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.
Why WIP? Is there work you still expect to do for this to be a complete contribution?
Please add an Enhancement entry to the change log at doc/whats_new/v0.22.rst
. Like the other entries there, please reference this pull request with :pr:
and credit yourself (and other contributors if applicable) with :user:
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. Do you still have changes to add to this PR in mind? If not you can rename the title of the PR from [WIP] (Work in Progress) to [MRG] (ready to merge).
Hi, would it be useful to do the same on the |
Yes, pr welcome for predict kwargs
|
Here's my attempt to help you: #14992 |
Reference Issues/PRs
Fixes #13349
What does this implement/fix? Explain your changes.
Originally TransformedTargetRegressor only passed
sample_weight
to thefit
method of the underlying regressor. However This regressor might have other parameters and could theoretically even be a pipeline. With this change you can pass arbitrary paramter to it.