CI: Fix tests broken by np 1.18 sorting change #29877
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Longer-term, do we want to use numpy's new behavior?
cc @TomAugspurger
Update A little more background: consider two ndarrays in np<1.18
In numpy 1.18, the behavior of
np.sort(arr2)
is changing to put the NaT at the end instead of at the beginning, to behave more like NaN. This breaks a few tests, which this PR fixes.Side-note: as of now, 1.18 changes the behavior for datetime64, but not timedelta64.