Closed jmert closed 4 years ago
Merging #17 into master will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #17 +/- ##
=======================================
Coverage 95.52% 95.52%
=======================================
Files 9 9
Lines 246 246
=======================================
Hits 235 235
Misses 11 11
Impacted Files | Coverage Δ | |
---|---|---|
src/interface.jl | 66.66% <ø> (ø) |
|
src/norm_sphere.jl | 100.00% <ø> (ø) |
|
src/norm_unit.jl | 100.00% <ø> (ø) |
|
src/calculation.jl | 95.72% <100.00%> (ø) |
|
src/norm_table.jl | 94.28% <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 d08c4b0...eb47615. Read the comment docs.
I went with renaming to
Legendre.coeff_?
instead of the redundantLegendre.legendre_?
as suggested in the issue, andPlm_00
is being renamed toinitcond
.Fixes #7