Closed joergi-w closed 3 years ago
Merging #171 (e180d2a) into master (d14e155) will increase coverage by
0.31%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #171 +/- ##
==========================================
+ Coverage 96.48% 96.80% +0.31%
==========================================
Files 19 19
Lines 797 814 +17
==========================================
+ Hits 769 788 +19
+ Misses 28 26 -2
Impacted Files | Coverage Δ | |
---|---|---|
src/iGenVar.cpp | 100.00% <ø> (+1.65%) |
:arrow_up: |
src/variant_detection/snp_indel_detection.cpp | 98.48% <100.00%> (+0.92%) |
:arrow_up: |
src/variant_detection/variant_detection.cpp | 93.97% <0.00%> (ø) |
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 d14e155...e180d2a. Read the comment docs.
I'm using the activity values from a previous PR to calculate the regions of high activity with a sliding window approach. The regions are returned in the format of intervals (i.e. pairs of genome positions).
The first commit adds the new functionality, the second commit structures the already quite long function into smaller, testable functions. The third commit adds API tests in order to satisfy coverage tests. Therefore, it may be easier to review them separately.
Resolves #153.