Extend HDF5 compression options
Closes #236 by expanding the hdf5 compression options to include lzf and also szip in addition to the standard gzip default options (which are integers for the compression level). As such the variable is renamed from compression_level to compression_option.
- The default compression option in
get_h5_literalsis changed to4to match that of the h5py library. - As the matlab function
h5createcan not write withlzfcompression a test is created in h5io_test - szip is not tested as it must be installed separately.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 67.50%. Comparing base (
d8f56f3) to head (4b1a563).
Additional details and impacted files
@@ Coverage Diff @@
## master #332 +/- ##
==========================================
+ Coverage 67.42% 67.50% +0.08%
==========================================
Files 48 48
Lines 7024 7036 +12
Branches 1596 1595 -1
==========================================
+ Hits 4736 4750 +14
+ Misses 1700 1699 -1
+ Partials 588 587 -1
| Flag | Coverage Δ | |
|---|---|---|
| 3.10 | 67.66% <100.00%> (+0.08%) |
:arrow_up: |
| 3.11 | 67.66% <100.00%> (+0.08%) |
:arrow_up: |
| 3.12 | 67.66% <100.00%> (+0.08%) |
:arrow_up: |
| 3.9 | 67.48% <100.00%> (+0.08%) |
:arrow_up: |
| ubuntu-latest | 67.45% <100.00%> (+0.08%) |
:arrow_up: |
| windows-latest | 67.45% <100.00%> (+0.08%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I think this is a great start, but still needs some work before being merged in.