[MRG+1] Fix the behavior of astype copy=True#4555
[MRG+1] Fix the behavior of astype copy=True#4555jnothman merged 2 commits intoscikit-learn:masterfrom
Conversation
There was a problem hiding this comment.
Does travis contain the old version?
There was a problem hiding this comment.
I think so. I don't have a numpy 1.6.1 at hand right now to quickly check.
There was a problem hiding this comment.
I did the check on numpy 1.6.1 with docker. numpy 1.6.1 is the version of numpy shipped with Ubuntu 12.04 and that is the oldest version of numpy supported by sklearn) and I got:
>>> a.astype(np.float32, copy=False)
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
TypeError: astype() takes no keyword arguments
we have a travis config for that. So this case is properly tested.
|
Apart from my comment LGTM. |
ogrisel
changed the title
ogrisel
changed the title
ogrisel
force-pushed
the
fix-astype-copy-true
branch
from
91fac5a to
95fb92b
Compare
|
@amueller I forced pushed a change to use a |
|
Cool. LGTM. |
|
@jnothman any time for a quick review on this tiny tiny fix on your code? |
|
Random ping to get this fix in @arjoly @glouppe @mblondel @pprett @agramfort @GaelVaroquaux. |
|
LGTM! Thanks for the fix :s |
[MRG] Fix the behavior of astype copy=True
|
Thanks :) |
3 participants
The astype backport should copy memory by default to be consistent with the default behavior of recent numpy.