TutteInstitute / vectorizers

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

Linear Optimal Transport #52

Closed lmcinnes closed 3 years ago

lmcinnes commented 3 years ago

Add linear optimal transport. Start some refactoring: move the KDEVectorizer to a separate file. Reblack everything to clean up formatting.

codecov-io commented 3 years ago

Codecov Report

Merging #52 (06e835a) into master (d15de4d) will increase coverage by 12.78%. The diff coverage is 65.92%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #52       +/-   ##
===========================================
+ Coverage   57.79%   70.57%   +12.78%     
===========================================
  Files          11       12        +1     
  Lines        2438     2277      -161     
===========================================
+ Hits         1409     1607      +198     
+ Misses       1029      670      -359     
Impacted Files Coverage Δ
vectorizers/_vectorizers.py 83.98% <ø> (-0.13%) :arrow_down:
vectorizers/_window_kernels.py 25.31% <ø> (ø)
vectorizers/distances.py 8.02% <ø> (-0.34%) :arrow_down:
vectorizers/em_token_cooccurrence.py 75.60% <ø> (ø)
vectorizers/utils.py 54.44% <ø> (ø)
vectorizers/linear_optimal_transport.py 58.33% <58.33%> (ø)
vectorizers/kde_vectorizer.py 87.09% <87.09%> (ø)
vectorizers/tests/test_common.py 98.63% <97.56%> (+0.12%) :arrow_up:
vectorizers/__init__.py 100.00% <100.00%> (ø)
... and 1 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 d15de4d...06e835a. Read the comment docs.

jc-healy commented 3 years ago

All test successful. Merging this lovely PR now.