-
-
Notifications
You must be signed in to change notification settings - Fork 25.3k
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
[MRG] FIX change boolean array-likes indexing in old NumPy version #14510
Conversation
@jnothman @amueller @jorisvandenbossche It seems that the I don't know if this is the right fix thought |
This seems the correct fix to me. But no plans to bump the numpy version in the next release? (which might make this unneeded) |
Not if we keep synchronized with the Ubuntu LTS support. |
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, thought I'm not sure whether it's for version 1.12 or 1.13
Thanks @glemaitre
closes #13858
Boolean array-likes and boolean arrays are not indexing the same way for older NumPy version.
Up-to-now it was not an issue, but it seems that we have a bug in
ColumnTransformer
.We should make sure that boolean array-likes are converted to arrays for older NumPy version.