root-11 / tablite

multiprocessing enabled out-of-memory data analysis library for tabular data.
MIT License
37 stars 8 forks source link

Fix xls get_headers #75

Closed realratchet closed 1 year ago

realratchet commented 1 year ago

Fix issues with get_headers falling back to text reading when reading 0 lines of excel, fix issue where reading excel file would ignore file count, excel file reader now has parity for linecount selection.

codecov-commenter commented 1 year ago

Codecov Report

Merging #75 (e7f9a81) into master (53bb033) will decrease coverage by 0.02%. Report is 3 commits behind head on master. The diff coverage is 87.50%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   89.30%   89.29%   -0.02%     
==========================================
  Files          49       49              
  Lines        6706     6709       +3     
==========================================
+ Hits         5989     5991       +2     
- Misses        717      718       +1     
Flag Coverage Δ
pytest 89.29% <87.50%> (-0.02%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
tablite/file_reader_utils.py 85.78% <85.71%> (-0.30%) :arrow_down:
tablite/version.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more