Skip to content

TST: test_read_clipboard_infer_excel warning #19010

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

Closed
jreback opened this issue Dec 30, 2017 · 5 comments
Closed

TST: test_read_clipboard_infer_excel warning #19010

jreback opened this issue Dec 30, 2017 · 5 comments
Labels
Output-Formatting __repr__ of pandas objects, to_string Testing pandas testing functions or related to the test suite
Milestone

Comments

@jreback
Copy link
Contributor

jreback commented Dec 30, 2017

pandas/tests/io/test_clipboard.py::TestClipboard::()::test_read_clipboard_infer_excel
  /home/travis/build/jreback/pandas/pandas/io/clipboards.py:63: ParserWarning: Falling back to the 'python' engine because the 'c' engine does not support regex separators (separators > 1 char and different from '\s+' are interpreted as regex); you can avoid this warning by specifying engine='python'.
    return read_table(StringIO(text), sep=sep, **kwargs)
@jreback jreback added Output-Formatting __repr__ of pandas objects, to_string Testing pandas testing functions or related to the test suite labels Dec 30, 2017
@jreback jreback added this to the 0.23.0 milestone Dec 30, 2017
@jreback
Copy link
Contributor Author

jreback commented Dec 30, 2017

cc @gfyoung if you'd take a look

@gfyoung
Copy link
Member

gfyoung commented Dec 31, 2017

There are two ways to fix this:

  1. Pass in engine='python'
  2. Assert that a warning gets issued (out of our control if their data requires multi-char separator and their defaulting to engine='c' in `read_table)

Thoughts?

@jreback
Copy link
Contributor Author

jreback commented Dec 31, 2017

yeah I think the engine='python' default may be ok here to avoid the warning, though maybe only need to do this if we are passing in a non-whitespace delimiter.

@gfyoung
Copy link
Member

gfyoung commented Jan 1, 2018

yeah I think the engine='python' default

I don't think the default should be changed. Rather, we should just pass in engine='python' for the test.

@jreback
Copy link
Contributor Author

jreback commented Jan 1, 2018

sgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Output-Formatting __repr__ of pandas objects, to_string Testing pandas testing functions or related to the test suite
Projects
None yet
Development

No branches or pull requests

2 participants