-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CI: Fix npdev build #29860
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
CI: Fix npdev build #29860
Conversation
@jreback @TomAugspurger npdev build is green here |
@@ -765,13 +765,15 @@ def test_concat_join_axes_deprecated(self, axis): | |||
) | |||
|
|||
expected = pd.concat([one, two], axis=1, sort=False).reindex(index=two.index) | |||
result = pd.concat([one, two], axis=1, sort=False, join_axes=[two.index]) | |||
with tm.assert_produces_warning(FutureWarning): |
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.
Where are these warnings coming from?
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.
join_axes is deprecated
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.
And this failure wasn't caught earlier because the numpydev build is the only one that fails on warnings, and that was offline for a bit? If so, feel free to merge.
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.
That's my understanding, yes
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.
join_axes
is already deprecated for a few months, so there's probably another reason ..
thanks! |
@jbrockmendel do you know if this are deliberate changes on numpy's side, or is this something to report upstream? |
not that im aware of |
No description provided.