Closed LTLA closed 3 years ago
Merging #2 (c5e5699) into master (575ea35) will decrease coverage by
0.22%
. The diff coverage is95.12%
.
@@ Coverage Diff @@
## master #2 +/- ##
==========================================
- Coverage 97.87% 97.64% -0.23%
==========================================
Files 6 9 +3
Lines 518 510 -8
==========================================
- Hits 507 498 -9
- Misses 11 12 +1
Impacted Files | Coverage Δ | |
---|---|---|
include/qdtsne/gaussian.hpp | 0.00% <0.00%> (ø) |
|
include/qdtsne/utils.hpp | 78.57% <ø> (ø) |
|
include/qdtsne/tsne.hpp | 97.32% <92.85%> (+0.41%) |
:arrow_up: |
tests/src/gaussian.cpp | 100.00% <100.00%> (ø) |
|
tests/src/symmetrize.cpp | 100.00% <100.00%> (ø) |
|
tests/src/tsne.cpp | 97.43% <100.00%> (-0.24%) |
: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 575ea35...c5e5699. Read the comment docs.
Need to add more rigorous tests for symmetrization; plus, check that the entropy is within tolerance.
Improved the symmetrization to avoid K^2 search.