Closed zsteve closed 2 years ago
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
src/utils.jl | 1 | 2 | 50.0% | ||
<!-- | Total: | 1 | 2 | 50.0% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
src/utils.jl | 1 | 95.35% | ||
<!-- | Total: | 1 | --> |
Totals | |
---|---|
Change from base Build 1260631313: | 10.0% |
Covered Lines: | 590 |
Relevant Lines: | 619 |
Merging #146 (cee80a6) into master (803274d) will increase coverage by
10.03%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #146 +/- ##
===========================================
+ Coverage 85.27% 95.31% +10.03%
===========================================
Files 14 14
Lines 618 619 +1
===========================================
+ Hits 527 590 +63
+ Misses 91 29 -62
Impacted Files | Coverage Δ | |
---|---|---|
src/utils.jl | 95.34% <100.00%> (+2.32%) |
:arrow_up: |
src/entropic/symmetric.jl | 96.42% <0.00%> (+96.42%) |
:arrow_up: |
src/entropic/sinkhorn_divergence.jl | 100.00% <0.00%> (+100.00%) |
:arrow_up: |
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 803274d...cee80a6. Read the comment docs.
I am clearly a donkey and didn't include sinkhorn_divergence in
runtests.jl
(the tests all pass, though). will merge directly