TutteInstitute / vectorizers

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

Refactored kernels and windows #22

Closed cjweir closed 4 years ago

cjweir commented 4 years ago

SkipGramVectorizer and TokenCoocurrenceVectorizer now allow for string named kernel and window functions which were moved into a new seperate file. Instead of args, the classes now simply take window_radius.

codecov-io commented 4 years ago

Codecov Report

Merging #22 into master will increase coverage by 0.81%. The diff coverage is 85.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage   63.64%   64.45%   +0.81%     
==========================================
  Files           9       10       +1     
  Lines        1656     1722      +66     
==========================================
+ Hits         1054     1110      +56     
- Misses        602      612      +10     
Impacted Files Coverage Δ
vectorizers/_vectorizers.py 86.55% <67.74%> (-1.88%) :arrow_down:
vectorizers/_window_kernels.py 100.00% <100.00%> (ø)
vectorizers/tests/test_common.py 99.48% <100.00%> (+<0.01%) :arrow_up:
vectorizers/distances.py 81.72% <0.00%> (+0.99%) :arrow_up:
vectorizers/optimal_transport.py 10.44% <0.00%> (+2.40%) :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 5dd9b34...1159fcb. Read the comment docs.