TutteInstitute / vectorizers

Vectorizers for a range of different data types
BSD 3-Clause "New" or "Revised" License
97 stars 23 forks source link

Fix up some of the test coverage. #82

Closed lmcinnes closed 3 years ago

codecov-commenter commented 3 years ago

Codecov Report

Merging #82 (30662a7) into master (644d698) will increase coverage by 1.43%. The diff coverage is 96.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
+ Coverage   90.27%   91.70%   +1.43%     
==========================================
  Files          19       19              
  Lines        3907     3870      -37     
  Branches      747        0     -747     
==========================================
+ Hits         3527     3549      +22     
+ Misses        324      321       -3     
+ Partials       56        0      -56     
Impacted Files Coverage Δ
vectorizers/linear_optimal_transport.py 92.95% <50.00%> (+2.42%) :arrow_up:
vectorizers/tests/test_common.py 99.83% <100.00%> (+<0.01%) :arrow_up:
vectorizers/tests/test_transformers.py 100.00% <100.00%> (ø)
vectorizers/transformers.py 95.42% <100.00%> (+2.03%) :arrow_up:
vectorizers/utils.py 73.60% <0.00%> (-1.28%) :arrow_down:
vectorizers/kde_vectorizer.py 91.93% <0.00%> (-0.13%) :arrow_down:
vectorizers/ngram_vectorizer.py 86.44% <0.00%> (+0.84%) :arrow_up:
vectorizers/coo_utils.py 92.22% <0.00%> (+1.26%) :arrow_up:
... and 5 more

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 644d698...30662a7. Read the comment docs.

lmcinnes commented 3 years ago

So something is weird here, and travis and coveralls didn't run, so ... um ... what?