search

kevlar-dev / kevlar

Reference-free variant discovery in large eukaryotic genomes
https://kevlar.readthedocs.io
MIT License
41 stars 9 forks source link

Fix bug with merging variants #337

Closed standage closed 5 years ago

standage commented 5 years ago

In #332, a test_merge method was added to the Variant class to see if two consecutive variant predictions are adjacent and mergeable into a multinucleotide variant. The new function did not account for scenarios in which a false call may have no flanking windows. This update corrects this bug.

codecov[bot] commented 5 years ago

Codecov Report

Merging #337 into master will decrease coverage by 0.13%. The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #337      +/-   ##
==========================================
- Coverage   95.13%   95.01%   -0.13%     
==========================================
  Files          48       48              
  Lines        3000     3004       +4     
  Branches      561      563       +2     
==========================================
  Hits         2854     2854              
- Misses        106      108       +2     
- Partials       40       42       +2
Impacted Files Coverage Δ
kevlar/vcf.py 95.94% <0%> (-1.44%) :arrow_down:

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 5ead836...506efd0. Read the comment docs.