molgenis / VaSeBuilder

Validation Set Builder
GNU Lesser General Public License v3.0
1 stars 3 forks source link

BVCS bug in overlapping contexts #65

Closed TDMedina closed 4 years ago

TDMedina commented 5 years ago

VaSeBuilder.py, Line 1849:

Overlapping context (line 1894-1896) will return NoneType at line 1846. Line 1849 contains debug message calling vcontext.get_variant_context_chrom() which does not exist for NoneType.

Need to check object type and return early with alternate message if contexts overlap at this point, I think.

TDMedina commented 5 years ago

Still a bug in release 19.10.1. However, these same-sample overlapping contexts will probably be merged into combined 2-variant contexts in future, which will change / eliminate this bug.