Skip to content

Parquet: Add error message for no engine found #18717

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 2 commits into from
Dec 10, 2017

Conversation

jorisvandenbossche
Copy link
Member

Give a better error message for engine="auto" case when none of the
engines were found installed.

closes #18156

Cherry-picking the doc part of PR #18568

Give a better error message for engine="auto" case when none of the
engines were found installed.
@pep8speaks
Copy link

pep8speaks commented Dec 10, 2017

Hello @jorisvandenbossche! Thanks for updating the PR.

Cheers ! There are no PEP8 issues in this Pull Request. 🍻

Comment last updated on December 10, 2017 at 16:47 Hours UTC

@jorisvandenbossche jorisvandenbossche added the IO Parquet parquet, feather label Dec 10, 2017
@jorisvandenbossche jorisvandenbossche added the Error Reporting Incorrect or improved errors from pandas label Dec 10, 2017
@jorisvandenbossche jorisvandenbossche added this to the 0.21.1 milestone Dec 10, 2017
@codecov
Copy link

codecov bot commented Dec 10, 2017

Codecov Report

Merging #18717 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #18717      +/-   ##
==========================================
- Coverage   91.58%   91.56%   -0.02%     
==========================================
  Files         153      153              
  Lines       51305    51308       +3     
==========================================
- Hits        46989    46982       -7     
- Misses       4316     4326      +10
Flag Coverage Δ
#multiple 89.43% <0%> (-0.01%) ⬇️
#single 40.71% <0%> (-0.1%) ⬇️
Impacted Files Coverage Δ
pandas/io/parquet.py 64.55% <0%> (-0.83%) ⬇️
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/frame.py 97.81% <0%> (-0.1%) ⬇️
pandas/io/pytables.py 92.84% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c1f507...97b861f. Read the comment docs.

@jreback
Copy link
Contributor

jreback commented Dec 10, 2017

sure

@jreback jreback merged commit b8b108c into pandas-dev:master Dec 10, 2017
@jorisvandenbossche jorisvandenbossche mentioned this pull request Dec 10, 2017
4 tasks
TomAugspurger pushed a commit to TomAugspurger/pandas that referenced this pull request Dec 11, 2017
* patquet.py: Add error message for no engine found

Give a better error message for engine="auto" case when none of the
engines were found installed.

* clean-up to prevent merge conflicts with other PR

(cherry picked from commit b8b108c)
TomAugspurger pushed a commit that referenced this pull request Dec 12, 2017
* patquet.py: Add error message for no engine found

Give a better error message for engine="auto" case when none of the
engines were found installed.

* clean-up to prevent merge conflicts with other PR

(cherry picked from commit b8b108c)
yarikoptic added a commit to neurodebian/pandas that referenced this pull request Feb 22, 2018
Version 0.22.0

* tag 'v0.22.0': (777 commits)
  RLS: v0.22.0
  DOC: Fix min_count docstring (pandas-dev#19005)
  DOC: More 0.22.0 updates (pandas-dev#19002)
  TST: Remove pow test in expressions
  COMPAT: Avoid td.skip decorator
  DOC: 0.22.0 release docs (pandas-dev#18983)
  DOC: Include 0.22.0 whatsnew
  Breaking changes for sum / prod of empty / all-NA (pandas-dev#18921)
  ENH: Added a min_count keyword to stat funcs (pandas-dev#18876)
  RLS: v0.21.1
  DOC: Add date to whatsnew (pandas-dev#18740)
  DOC: Include 0.21.1 whatsnew
  DOC: Update relase notes (pandas-dev#18739)
  CFG: Ignore W503
  DOC: fix options table (pandas-dev#18730)
  ENH: support non default indexes in writing to Parquet (pandas-dev#18629)
  BUG: Fix to_latex with longtable (pandas-dev#17959) (pandas-dev#17960)
  Parquet: Add error message for no engine found (pandas-dev#18717)
  BUG: Categorical data fails to load from hdf when all columns are NaN (pandas-dev#18652)
  DOC: clean-up whatsnew file for 0.21.1 (pandas-dev#18690)
  ...
yarikoptic added a commit to neurodebian/pandas that referenced this pull request Feb 22, 2018
* releases: (777 commits)
  RLS: v0.22.0
  DOC: Fix min_count docstring (pandas-dev#19005)
  DOC: More 0.22.0 updates (pandas-dev#19002)
  TST: Remove pow test in expressions
  COMPAT: Avoid td.skip decorator
  DOC: 0.22.0 release docs (pandas-dev#18983)
  DOC: Include 0.22.0 whatsnew
  Breaking changes for sum / prod of empty / all-NA (pandas-dev#18921)
  ENH: Added a min_count keyword to stat funcs (pandas-dev#18876)
  RLS: v0.21.1
  DOC: Add date to whatsnew (pandas-dev#18740)
  DOC: Include 0.21.1 whatsnew
  DOC: Update relase notes (pandas-dev#18739)
  CFG: Ignore W503
  DOC: fix options table (pandas-dev#18730)
  ENH: support non default indexes in writing to Parquet (pandas-dev#18629)
  BUG: Fix to_latex with longtable (pandas-dev#17959) (pandas-dev#17960)
  Parquet: Add error message for no engine found (pandas-dev#18717)
  BUG: Categorical data fails to load from hdf when all columns are NaN (pandas-dev#18652)
  DOC: clean-up whatsnew file for 0.21.1 (pandas-dev#18690)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Error Reporting Incorrect or improved errors from pandas IO Parquet parquet, feather
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Minor doc issue in to_parquet(engine='auto')
5 participants