CTUAvastLab / Mill.jl

Build flexible hierarchical multi-instance learning models.
https://ctuavastlab.github.io/Mill.jl/stable/
MIT License
86 stars 8 forks source link

Dev PR #116

Closed simonmandlik closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Merging #116 (a012c3d) into master (30a61f0) will increase coverage by 0.08%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
+ Coverage   91.12%   91.20%   +0.08%     
==========================================
  Files          36       36              
  Lines        1667     1490     -177     
==========================================
- Hits         1519     1359     -160     
+ Misses        148      131      -17     
Flag Coverage Δ
unittests 91.20% <100.00%> (+0.08%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/modelnodes/bagmodel.jl 83.33% <ø> (ø)
src/util.jl 89.79% <ø> (ø)
src/aggregations/segmented_lse.jl 100.00% <100.00%> (+12.50%) :arrow_up:
src/aggregations/segmented_pnorm.jl 100.00% <100.00%> (+2.88%) :arrow_up:
src/aggregations/segmented_sum.jl 100.00% <100.00%> (ø)
src/gradients.jl 94.91% <100.00%> (-0.09%) :arrow_down:
src/special_arrays/special_arrays.jl 93.75% <100.00%> (-0.37%) :arrow_down:

... and 12 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

simonmandlik commented 1 year ago

It is already mentioned just below

https://github.com/CTUAvastLab/Mill.jl/blob/30a61f0c0e8ed045cb0612a28ecf89f1b4c09b3c/docs/make.jl#L16-L18

I personally prefer the Python version because it is already included with most python versions and LiveServer.jl is another dependence. But it doesn't really matter, it's just for local testing of docs.