broadinstitute / adapt

A package for designing activity-informed nucleic acid diagnostics for viruses.
MIT License
27 stars 1 forks source link

Add per sequence analysis to analyze coverage; generalize analyze coverage and pick test targets to work with PCR #59

Closed priyappillai closed 2 years ago

priyappillai commented 2 years ago

Add the following features to analyze_coverage.py:

Also, allow pick_test_targets.py to work with variable primer/guide lengths

Details

In coverage_analysis.py

In test_coverage_analysis.py

In analyze_coverage.py

Add test_analyze_coverage.py

In guide.py

In seq_io.py

In test_design.py

codecov[bot] commented 2 years ago

Codecov Report

Merging #59 (c371c12) into main (a96ce8d) will increase coverage by 1.16%. The diff coverage is 95.08%.

@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   85.15%   86.31%   +1.16%     
==========================================
  Files          48       50       +2     
  Lines        7599     8171     +572     
==========================================
+ Hits         6471     7053     +582     
+ Misses       1128     1118      -10     
Impacted Files Coverage Δ
adapt/utils/seq_io.py 32.84% <50.00%> (+0.66%) :arrow_up:
bin/analyze_coverage.py 93.13% <91.39%> (ø)
bin/tests/test_analyze_coverage.py 93.42% <93.42%> (ø)
adapt/coverage_analysis.py 97.16% <96.72%> (+13.91%) :arrow_up:
adapt/tests/test_coverage_analysis.py 99.31% <100.00%> (+12.64%) :arrow_up:
adapt/utils/guide.py 100.00% <100.00%> (ø)
bin/tests/test_design.py 99.51% <100.00%> (ø)
... and 1 more

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 a96ce8d...c371c12. Read the comment docs.