Closed nalimilan closed 3 years ago
Merging #85 (e4a068d) into master (54f9b0d) will decrease coverage by
1.28%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #85 +/- ##
==========================================
- Coverage 98.18% 96.89% -1.29%
==========================================
Files 1 1
Lines 386 419 +33
==========================================
+ Hits 379 406 +27
- Misses 7 13 +6
Impacted Files | Coverage Δ | |
---|---|---|
src/Statistics.jl | 96.89% <100.00%> (-1.29%) |
:arrow_down: |
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 54f9b0d...e4a068d. Read the comment docs.
cov(::AbstractVector, ::AbstractVector)
was less stable than othercov
methods and thanvar
on arrays as it calledsum
on a generator, which does not use pairwise summation. Use aBroadcasted
object instead to benefit from pairwise summation.Fixes https://github.com/JuliaLang/Statistics.jl/issues/83.
Performance also seems to increase a bit: