-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
COMPAT: breakage with just merged: https://github.com/numpy/numpy/pull/7325 #12687
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
Comments
@gfyoung I think we just need to also accept |
@jreback : The test needs to be modified to check which version of |
I don't think its a test failure, rather need to accept
|
Fair enough. I'll tackle this issue in a PR that will also fix #12644. |
@gfyoung if you have a chance on this one. annoying when master fails :< (even though its the allowed failures, that's just to make it finish faster :) |
@jreback : No worries. As I said above, I'll fix it in a PR that will address all of the |
@gfyoung ok, great. |
@gfyoung if you are looking for something to do.....I want to fix the failing build on travis (numpy compat build) for this (can leave open for more general things). |
@jreback : Alas, the fix is more complicated than just adding xref #7491 in |
My massive PR #12810 will address this breakage on the |
Does this fix the pandas breakage without any work on the pandas side? I'm somewhat annoyed that there was a problem. OOPS: wrong PR ;) |
Augment pandas array-like methods with appropriate parameters (generally, '*args' and '**kwargs') so that they can be called via analogous functions in the numpy library they are defined in 'fromnumeric.py'. Closes pandas-devgh-12638. Closes pandas-devgh-12644. Closes pandas-devgh-12687.
see here
xref #12603
#12644
numpy issue 7325
cc @gfyoung
cc @njsmith
The text was updated successfully, but these errors were encountered: