-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
BUG: Failing eval tests on OS X #5118
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
@jtratner want to take this one? happy to help .... i just don't have a mac to test on ... |
@TomAugspurger - do you see this when you run the eval test suite? (or even just the tp_compare error) |
I got these two warnings:
and the end of the output:
|
Well at least we have more evidence that it's an OSX-specific problem now (instead of just my setup). Thanks for running it! |
This was on pandas |
yeah ... not sure why it passes on my machine (linux box) since the sort fails on both linux and mac. let me take a look |
@cpcloud and we really have a more minimal example that will fail more clearly from pandas import *
tm = util.testing
with tm.assert_produces_warning(False):
try:
arr = np.array([Timestamp(n) for n in range(10)] + range(10))
arr.sort()
except:
pass |
@jreback @TomAugspurger I'm assuming that fails for you both, even though it passes on my end |
That fails like this for me:
|
@cpcloud any idea what's going on here? |
Last we left this, the issue was that it was doing a numpy inplace sort, but because the objects are not comparable it leaves it in an inconsistent state between architectures and sort methods (not a numpy bug, just the result of sorting inplace and raising an Exception at some point). Should we just eliminate the test case, given that it doesn't really make sense? |
@cpcloud this is the combine a datetimeindex and an intindex. I thought we are raising on this? (unless you actually add them) |
It should raise, because it doesn't really make sense. |
@cpcloud ? |
is this still reproducing? |
this is the same as #5039 ... closing |
These two tests fail (note the warning that it generates, happens in one other place but the test passes).
Traceback:
The text was updated successfully, but these errors were encountered: