-
-
Notifications
You must be signed in to change notification settings - Fork 26k
FIX Preserve y shape in TransformedTargetRegressor #31563
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
base: main
Are you sure you want to change the base?
FIX Preserve y shape in TransformedTargetRegressor #31563
Conversation
❌ Linting issuesThis PR is introducing linting issues. Here's a summary of the issues. Note that you can avoid having linting issues by enabling You can see the details of the linting issues under the
|
Thanks for the Pull Request @kryggird! If you have the time, could you take a look at the linter's complaints - it has instructions on how to install and run the linters as well which in 99.9% will automagically fix all the complaints. |
Reference Issues/PRs
Fixes 26530.
What does this implement/fix? Explain your changes.
This PR uses the existing
self._training_dim
to decide whether to squeezey
before passing it the inner regressor inTransformedTargetRegressor
.Any other comments?
I've also added a test in
test_metaestimators_metadata_routing.py
.@betatim @glemaitre