-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Catch Pandas AssertionError on deprecated multidimensional indexing. Closes #18158 #18167
Conversation
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.
.... I guess 😉.
It might be better to narrow the AssertionError catch to the Pandas warning using: ...
try:
ndim = x[:, None].ndim
except AssertionError as exc:
if len(w) > 0:
return np.asanyarray(x)
else:
raise AssertionError(exc) from None
... Do you prefer this approach? |
I wasn't concerned about the approach, just that we have to do such a dance to get around it. If we are going to first-class pandas like this, we really need to test their RCs and vice-versa. |
…ed multidimensional indexing. Closes matplotlib#18158
Thanks @l-johnston ! I think (?) this is your first Matplotlib RP, 🎉 Hopefully we will hear from you again! |
PR Summary
Fixes #18158 due to pandas-dev/pandas#35527