-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
BUG: revert 2/3 changes in vbsuite #4478
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
Conversation
Can you take care of #4474 as well please? |
done. I just checked out vb_suite from 338f340 - I don't think there were any other changes aside from 2to3 related ones, so probably should be okay. You might want to confirm that (tried to look through the diff). I have to run but can look again tonight. |
@jreback this just popped up on Travis - any idea? Given that this only changes vb_suite, I'm pretty sure it's unrelated.
|
sometimes I see a random failure on travis (but can't reproduce it outside)....maybe a writing to disk error of some kind...if it reproduces let me know |
@jreback just happened twice in a row. |
@jreback trying it again, maybe it'll work this time... |
Actually every commit I've pushed based off of master:
etc. |
hmm...that is weird...let me pull down your branch |
works fine for me (both 32/64 bit) |
This commit is just current master failing with the same issue - https://travis-ci.org/jtratner/pandas/builds/9897047 |
are you sure you are completely rebased, I did push a fix last night to master (and in between I don't know what it would do) |
current master is fine |
@jreback can you checkout a branch off of master and force Travis to build? I keep getting issues with commits that are exactly the same as master. This time I deleted my master branch, checked out pydata/pandas@master and then push to my own branch - still failed. |
tell you what...why don't I just merge this branch....? |
@jreback haha, your call. But just be aware that this just failed about 2 minutes ago b/c of the same issue cropping up again :-/ |
BUG: revert 2/3 changes in vbsuite
let's see how master does....problem is I CAN't repoduce, nor could @cpcloud locally at all (even on a box exactl like travis), 32-bit, same exact versions of tables,numexpr,numpy,python....really odd |
ok....failed (same test)...let me see |
ok...its fixed, but I all I did was this (which I am now going to take out)....maybe its some weird whitespace issue???? |
@jtratner master looks good now....very weird error |
Fixes #4477, #4474