-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
BUG Fixes FunctionTransformer validation in inverse_transform #20961
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
BUG Fixes FunctionTransformer validation in inverse_transform #20961
Conversation
We also need to add in the documentation, the fitted attribute. However, it is only set if |
Oh no, I see that actually, we don't want it to exist then. So we only need to add it to the documentation. |
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 for the fix.
@glemaitre merge? |
Tagging for #20965 if it gets in. If it does not get it, I think this PR can also be considered a 1.0.1 bug fix |
Looks good. |
This PR fixes a bug in
FunctionTransformer
whereinverse_transform
storesn_features_in_
based on the input toinverse_transform
.