search

cldf / csvw

CSV on the web
Apache License 2.0
37 stars 6 forks source link

Support reading and writing CSV from and to zip files #45

Closed xrotwang closed 4 years ago

xrotwang commented 4 years ago

closes #44

codecov-io commented 4 years ago

Codecov Report

Merging #45 into master will increase coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   99.91%   99.91%           
=======================================
  Files          14       14           
  Lines        2238     2266   +28     
=======================================
+ Hits         2236     2264   +28     
  Misses          2        2
Impacted Files Coverage Δ
src/csvw/dsv.py 99.53% <100.00%> (ø)
src/csvw/metadata.py 100.00% <100.00%> (ø)
src/csvw/utils.py 100.00% <100.00%> (ø)
tests/test_metadata.py 100.00% <100.00%> (ø)

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 58be38c...2e240ab. Read the comment docs.

xrotwang commented 4 years ago

How's the performance with the 55mb FormTable? Noticeably slower?

chrzyki commented 4 years ago

No, I hardly notice a difference at all. Might be different for somewhat older/slower machines but I think that even for those the effect is negligible.