Skip to content

BUG: Throw ValueError when the format kwarg is passed to the HDFStore… #24155

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
Dec 9, 2018

Conversation

CianciuStyles
Copy link
Contributor

… constructor

@pep8speaks
Copy link

Hello @CianciuStyles! Thanks for submitting the PR.

@codecov
Copy link

codecov bot commented Dec 8, 2018

Codecov Report

Merging #24155 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #24155      +/-   ##
==========================================
+ Coverage    92.2%    92.2%   +<.01%     
==========================================
  Files         162      162              
  Lines       51700    51702       +2     
==========================================
+ Hits        47670    47672       +2     
  Misses       4030     4030
Flag Coverage Δ
#multiple 90.6% <50%> (-0.01%) ⬇️
#single 43.02% <100%> (ø) ⬆️
Impacted Files Coverage Δ
pandas/io/pytables.py 92.31% <100%> (ø) ⬆️

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 fc64ca8...3359cfd. Read the comment docs.

@codecov
Copy link

codecov bot commented Dec 8, 2018

Codecov Report

Merging #24155 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #24155      +/-   ##
==========================================
+ Coverage   92.21%   92.21%   +<.01%     
==========================================
  Files         162      162              
  Lines       51721    51723       +2     
==========================================
+ Hits        47692    47694       +2     
  Misses       4029     4029
Flag Coverage Δ
#multiple 90.61% <50%> (-0.01%) ⬇️
#single 43.03% <100%> (ø) ⬆️
Impacted Files Coverage Δ
pandas/io/pytables.py 92.31% <100%> (ø) ⬆️

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 0ac1197...fedddcf. Read the comment docs.

@@ -1492,6 +1492,7 @@ Notice how we now instead output ``np.nan`` itself instead of a stringified form
- Bug in :func:`DataFrame.to_string()` that broke column alignment when ``index=False`` and width of first column's values is greater than the width of first column's header (:issue:`16839`, :issue:`13032`)
- Bug in :func:`DataFrame.to_string()` that caused representations of :class:`DataFrame` to not take up the whole window (:issue:`22984`)
- Bug in :func:`DataFrame.to_csv` where a single level MultiIndex incorrectly wrote a tuple. Now just the value of the index is written (:issue:`19589`).
- :func:`HDFStore` will throw ``ValueError`` when the ``format`` kwarg is passed to the constructor (:issue:`13291`)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

throw -> raise

@jreback
Copy link
Contributor

jreback commented Dec 9, 2018

looks ok, ping when pushed / green.

@jreback jreback added API Design IO HDF5 read_hdf, HDFStore labels Dec 9, 2018
@jreback jreback added this to the 0.24.0 milestone Dec 9, 2018
@jreback
Copy link
Contributor

jreback commented Dec 9, 2018

lgtm. ping on green.

@CianciuStyles
Copy link
Contributor Author

Hello Jeff, it looks like everything is green now.

@jreback jreback merged commit 949b148 into pandas-dev:master Dec 9, 2018
@jreback
Copy link
Contributor

jreback commented Dec 9, 2018

thans @CianciuStyles

Pingviinituutti pushed a commit to Pingviinituutti/pandas that referenced this pull request Feb 28, 2019
Pingviinituutti pushed a commit to Pingviinituutti/pandas that referenced this pull request Feb 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Design IO HDF5 read_hdf, HDFStore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

format ignored in HDFStore.open
3 participants