-
Notifications
You must be signed in to change notification settings - Fork 587
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
No empty alts if you please [VS-918] #8326
Conversation
ef6244b
to
68b1072
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## ah_var_store #8326 +/- ##
================================================
Coverage ? 86.193%
Complexity ? 35521
================================================
Files ? 2192
Lines ? 166470
Branches ? 17918
================================================
Hits ? 143485
Misses ? 16601
Partials ? 6384 |
|
||
### Run GvsCreateVATfromVDS | ||
|
||
Note: in order for this workflow to run successfully the 'Use reference disks' option must be selected in Terra workflow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit
Note: in order for this workflow to run successfully the 'Use reference disks' option must be selected in Terra workflow | |
**Note:** in order for this workflow to run successfully the 'Use reference disks' option must be selected in Terra workflow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one small nit, feel free to ignore
fdfeccb
to
23a60c5
Compare
The empty alts that results from overtrimming in an alt allele UDF correlate strongly with VAT / alt allele mismatches, but from trial runs with and without this patch they don't seem to actually be causing the mismatches. Nonetheless I think this is a desirable change as it keeps VAT VIDs from getting weird. VIDs have the format
<contig>-<position>-<ref>-<alt>
; if the alt is empty the VID just ends in a dash which just seems like an edge case waiting to break something.