search

TutteInstitute / vectorizers

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

TimedTCV bug fix #102

Closed cjweir closed 2 years ago

cjweir commented 2 years ago

Fixed some of the preprocessing with TimedTCV and max_vocab (bug found at STAMP)

codecov-commenter commented 2 years ago

Codecov Report

Merging #102 (47d9835) into master (45751ca) will increase coverage by 0.35%. The diff coverage is 90.90%.

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage   90.50%   90.85%   +0.35%     
==========================================
  Files          32       32              
  Lines        4644     4649       +5     
==========================================
+ Hits         4203     4224      +21     
+ Misses        441      425      -16
Impacted Files Coverage Δ
vectorizers/timed_token_cooccurrence_vectorizer.py 98.90% <50.00%> (-1.10%) :arrow_down:
vectorizers/preprocessing.py 87.55% <100.00%> (+7.27%) :arrow_up:
vectorizers/tests/test_common.py 99.81% <100.00%> (+<0.01%) :arrow_up:
vectorizers/utils.py 78.31% <0.00%> (+1.20%) :arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more