-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
Fixed typo in test_column_transformer #14128
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
Merged
jnothman
merged 1 commit into
scikit-learn:master
from
lrjball:test_columntransformer_typo
Jun 20, 2019
Merged
Fixed typo in test_column_transformer #14128
jnothman
merged 1 commit into
scikit-learn:master
from
lrjball:test_columntransformer_typo
Jun 20, 2019
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
In test_column_transformer_dataframe(), X_df2 was created but never used. X_df was mistakely being used in its place. The PR fixes that typo.
rth
approved these changes
Jun 19, 2019
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.
Thanks. The original comment by Joris was in #14048 (comment)
jnothman
approved these changes
Jun 20, 2019
In test_column_transformer_dataframe(), X_df2 was created but never used. X_df was mistakely being used in its place. The PR fixes that typo.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Reference Issues/PRs
Was original part of #14048, but have separated it into its own PR.
What does this implement/fix? Explain your changes.
In test_column_transformer_dataframe(), X_df2 was created but never used. X_df was mistakenly used in its place. This PR fixes that typo.