noamteyssier / crispr_screen

Efficient Differential Expression and Gene Aggregation for CRISPR screens.
https://noamteyssier.github.io/crispr_screen/
MIT License
6 stars 0 forks source link

82 add testing on aggregation methods and clean up code #83

Closed noamteyssier closed 1 year ago

noamteyssier commented 1 year ago

Codecov Report

Merging #83 (abaf360) into main (f6b1341) will increase coverage by 8.07%. The diff coverage is 78.21%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
+ Coverage   64.25%   72.32%   +8.07%     
==========================================
  Files          30       30              
  Lines        1975     2219     +244     
==========================================
+ Hits         1269     1605     +336     
+ Misses        706      614      -92     
Impacted Files Coverage Δ
src/aggregation/compute_aggregation.rs 0.00% <0.00%> (-42.33%) :arrow_down:
src/aggregation/rra/alpha_rra.rs 0.00% <0.00%> (ø)
src/enrich/results.rs 89.28% <ø> (ø)
src/aggregation/utils.rs 93.22% <91.27%> (-6.78%) :arrow_down:
src/enrich/enrichment_testing.rs 84.47% <96.00%> (+1.95%) :arrow_up:
src/aggregation/mod.rs 100.00% <100.00%> (+100.00%) :arrow_up:
src/aggregation/mwu_inc/inc.rs 100.00% <100.00%> (+100.00%) :arrow_up:
src/aggregation/results.rs 100.00% <100.00%> (+100.00%) :arrow_up:
src/utils/agg.rs 100.00% <100.00%> (+12.90%) :arrow_up:
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.