search

JuliaAI / MLJText.jl

A an MLJ extension for accessing models and tools related to text analysis
MIT License
11 stars 1 forks source link

add BM25 and BagOfWords transformers, update tests, update readme, re… #12

Closed pazzo83 closed 2 years ago

pazzo83 commented 2 years ago

Adding two new transformers with updated tests and readme, Also some refactoring since there is some shared code now.

codecov-commenter commented 2 years ago

Codecov Report

Merging #12 (142752b) into dev (000a451) will increase coverage by 7.28%. The diff coverage is 89.86%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev      #12      +/-   ##
==========================================
+ Coverage   81.05%   88.33%   +7.28%     
==========================================
  Files           3        7       +4     
  Lines          95      180      +85     
==========================================
+ Hits           77      159      +82     
- Misses         18       21       +3
Impacted Files Coverage Δ
src/MLJText.jl 100.00% <ø> (ø)
src/abstract_text_transformer.jl 75.86% <75.86%> (ø)
src/bagofwords_transformer.jl 88.00% <88.00%> (ø)
src/bm25_transformer.jl 89.18% <89.18%> (ø)
src/utils.jl 97.67% <97.67%> (ø)
src/tfidf_transformer.jl 87.50% <100.00%> (+7.25%) :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 000a451...142752b. Read the comment docs.

storopoli commented 2 years ago

Yes I can, but only beginning next week (Final academic semester week is hell now).