Skip to content

Conversation

@jzwinck
Copy link
Contributor

@jzwinck jzwinck commented Aug 3, 2016

@jreback
Copy link
Contributor

jreback commented Aug 3, 2016

tests!

@jzwinck
Copy link
Contributor Author

jzwinck commented Aug 3, 2016

@jreback I added a test now (still just one commit).

@jreback
Copy link
Contributor

jreback commented Aug 3, 2016

lgtm. pls add a whatsnew note in Bug Fixes.

@jreback jreback added Bug IO HDF5 read_hdf, HDFStore labels Aug 3, 2016
@jreback jreback added this to the 0.19.0 milestone Aug 3, 2016
@jreback
Copy link
Contributor

jreback commented Aug 3, 2016

ping on green.

@jzwinck jzwinck force-pushed the fix-13884 branch 2 times, most recently from 859ffac to a861b34 Compare August 3, 2016 19:54
@codecov-io
Copy link

Current coverage is 85.29% (diff: 100%)

Merging #13888 into master will decrease coverage by <.01%

@@             master     #13888   diff @@
==========================================
  Files           139        139          
  Lines         50087      50088     +1   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits          42723      42723          
- Misses         7364       7365     +1   
  Partials          0          0          

Powered by Codecov. Last update 0be0d67...789fa59

@jzwinck
Copy link
Contributor Author

jzwinck commented Aug 4, 2016

@jreback ping, this is good to go.

@jreback
Copy link
Contributor

jreback commented Aug 4, 2016

thanks!

@jreback jreback closed this in 9c1e738 Aug 4, 2016
@jzwinck jzwinck deleted the fix-13884 branch August 4, 2016 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug IO HDF5 read_hdf, HDFStore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Round trip through HDF5 with format=table and localized DatetimeIndex discards index name

3 participants