BUG: Fix scalar methods to receive keyword arguments#9527
BUG: Fix scalar methods to receive keyword arguments#9527eric-wieser merged 8 commits intonumpy:masterfrom
Conversation
|
LGTM, although I wonder if there are others with this problem, and we should just use the kwargs path for all of them - which at worst is a (small) performance hit. |
|
@eric-wieser Thanks for your reply. |
|
Either way, this patch probably is good enough as is - just trying to start a discussion about future work |
| * | ||
| * #name = tolist, item, tostring, tobytes, astype, copy, __deepcopy__, | ||
| * #name = tolist, item, tostring, tobytes, copy, __deepcopy__, | ||
| * searchsorted, view, swapaxes, conj, conjugate, nonzero, flatten, |
There was a problem hiding this comment.
From looking at methods.c, these also don't belong here:
tostringtobytescopysearchsortedviewflattenravel
Many of these take order= arguments
|
@eric-wieser Fixed. |
eric-wieser
changed the title
|
Wow, you didn't need to do each of those in a separate commit - that's just making work for yourself! We can change PR titles easily from the web interface anyway, so don't worry about it - it's commit messages that are more work to change, but yours are fine already |
|
Separate commits did make it super-easy to review though, so thanks! |
The np.generic.astype method now accepts these keyword arguments
MAINT: Remove workarounds for gh-9527
The np.generic.astype method now accepts these keyword arguments
3 participants
This fixes #9512 .