JuliaIO / JLD2.jl

HDF5-compatible file format in pure Julia
Other
537 stars 84 forks source link

A few minor perforamnce fixes #470

Closed sjkelly closed 1 year ago

sjkelly commented 1 year ago
codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 33.33% and project coverage change: +1.88 :tada:

Comparison is base (30dd578) 84.04% compared to head (535df18) 85.92%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #470 +/- ## ========================================== + Coverage 84.04% 85.92% +1.88% ========================================== Files 31 31 Lines 4142 4156 +14 ========================================== + Hits 3481 3571 +90 + Misses 661 585 -76 ``` | [Impacted Files](https://app.codecov.io/gh/JuliaIO/JLD2.jl/pull/470?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaIO) | Coverage Δ | | |---|---|---| | [src/groups.jl](https://app.codecov.io/gh/JuliaIO/JLD2.jl/pull/470?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaIO#diff-c3JjL2dyb3Vwcy5qbA==) | `87.76% <0.00%> (+0.09%)` | :arrow_up: | | [src/compression.jl](https://app.codecov.io/gh/JuliaIO/JLD2.jl/pull/470?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaIO#diff-c3JjL2NvbXByZXNzaW9uLmps) | `91.30% <100.00%> (+1.40%)` | :arrow_up: | | [src/misc.jl](https://app.codecov.io/gh/JuliaIO/JLD2.jl/pull/470?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaIO#diff-c3JjL21pc2Muamw=) | `80.21% <100.00%> (+3.39%)` | :arrow_up: | ... and [11 files with indirect coverage changes](https://app.codecov.io/gh/JuliaIO/JLD2.jl/pull/470/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaIO)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.