Skip to content

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

Merged
merged 1 commit into from
Aug 7, 2013

Conversation

jtratner
Copy link
Contributor

@jtratner jtratner commented Aug 6, 2013

Fixes #4477, #4474

@ghost
Copy link

ghost commented Aug 6, 2013

Can you take care of #4474 as well please?

@jtratner
Copy link
Contributor Author

jtratner commented Aug 6, 2013

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.

@jtratner
Copy link
Contributor Author

jtratner commented Aug 6, 2013

@jreback this just popped up on Travis - any idea? Given that this only changes vb_suite, I'm pretty sure it's unrelated.

======================================================================
ERROR: test_string_select (pandas.io.tests.test_pytables.TestHDFStore)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/virtualenv/python2.7_with_system_site_packages/local/lib/python2.7/site-packages/pandas-0.11.0_1052_g7ad2e48-py2.7-linux-x86_64.egg/pandas/io/tests/test_pytables.py", line 2424, in test_string_select
    result = store.select('df',Term('x!=none'))
  File "/home/travis/virtualenv/python2.7_with_system_site_packages/local/lib/python2.7/site-packages/pandas-0.11.0_1052_g7ad2e48-py2.7-linux-x86_64.egg/pandas/io/pytables.py", line 475, in select
    raise KeyError('No object named %s in the file' % key)
KeyError: 'No object named df in the file'
----------------------------------------------------------------------

@jreback
Copy link
Contributor

jreback commented Aug 6, 2013

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

@jtratner
Copy link
Contributor Author

jtratner commented Aug 6, 2013

@jreback just happened twice in a row.

@jtratner
Copy link
Contributor Author

jtratner commented Aug 6, 2013

@jreback trying it again, maybe it'll work this time...

@jreback
Copy link
Contributor

jreback commented Aug 6, 2013

hmm...that is weird...let me pull down your branch

@jreback
Copy link
Contributor

jreback commented Aug 6, 2013

works fine for me (both 32/64 bit)

@jtratner
Copy link
Contributor Author

jtratner commented Aug 6, 2013

This commit is just current master failing with the same issue - https://travis-ci.org/jtratner/pandas/builds/9897047

@jreback
Copy link
Contributor

jreback commented Aug 6, 2013

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)

@jreback
Copy link
Contributor

jreback commented Aug 6, 2013

current master is fine

@jtratner
Copy link
Contributor Author

jtratner commented Aug 7, 2013

@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.

@jreback
Copy link
Contributor

jreback commented Aug 7, 2013

@jtratner fyi...@cpcloud ran into the same problem....but noone can repro (even on a very travis like machine), then suddenly went away....weird

@jreback
Copy link
Contributor

jreback commented Aug 7, 2013

tell you what...why don't I just merge this branch....?

@jtratner
Copy link
Contributor Author

jtratner commented Aug 7, 2013

@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 :-/

jreback added a commit that referenced this pull request Aug 7, 2013
BUG: revert 2/3 changes in vbsuite
@jreback jreback merged commit 1fbce08 into pandas-dev:master Aug 7, 2013
@jreback
Copy link
Contributor

jreback commented Aug 7, 2013

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

@jtratner jtratner deleted the revert-vb-suite branch August 7, 2013 00:29
@jreback
Copy link
Contributor

jreback commented Aug 7, 2013

ok....failed (same test)...let me see

@jreback
Copy link
Contributor

jreback commented Aug 7, 2013

ok...its fixed, but I all I did was this (which I am now going to take out)....maybe its some weird whitespace issue????
f08fb89

@jreback
Copy link
Contributor

jreback commented Aug 7, 2013

@jtratner master looks good now....very weird error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2to3 refactor broke vbenches
2 participants