-
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
VAT cleanup and documentation #7531
Conversation
6e3fa2d
to
66acc54
Compare
665a52a
to
8eed433
Compare
e953f75
to
8831c01
Compare
clean up annotations comment out ones that will be used in the future remove annotations I dont want
8831c01
to
e6d0810
Compare
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.
two small nits, otherwise 👍🏻
|
||
## make a file of just the first 5 columns of the tsv | ||
## During normalization, sometimes duplicate variamts appear but with different calculations. This seems to be a bug in bcftools. For now we arre dropping all duplicate variants | ||
## The say in which this is done is a bit hamfisted and should be optimized in the future. |
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: typo?
## The say in which this is done is a bit hamfisted and should be optimized in the future. | |
## The way in which this is done is a bit hamfisted and should be optimized in the future. |
@@ -326,9 +333,11 @@ task AnnotateVCF { | |||
echo "Creating custom annotations" | |||
mkdir customannotations_dir | |||
CUSTOM_ANNOTATIONS_FOLDER="$PWD/customannotations_dir" | |||
## TODO this is a pass thru |
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: do you still need this comment?
d9f0b08
to
e0641b0
Compare
Additional cleanup on the VAT--specifically focused on the failing shards and optimizing the workflow with them in mind.
As a next step, this workflow will be split up into 3 sub-workflows to keep the failures from knocking over the remaining likely-successful shards