-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add --input_variant parameter for handling single variant #112
Conversation
Can you find why the smoke test failed. @jylee-bcm |
bin/generate_input_vcf.py
Outdated
##FORMAT=<ID=PL,Number=G,Type=Integer,Description="Normalized, Phred-scaled likelihoods for genotypes as defined in the VCF specification"> | ||
##FORMAT=<ID=SB,Number=4,Type=Integer,Description="Per-sample component statistics which comprise the Fisher's Exact Test to detect strand bias."> | ||
##FILTER=<ID=PASS,Description="All filters passed"> | ||
##reference=file:///staging/human/reference/b37/b37.fa.default/reference.bin |
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.
does it only handle hg19? not hg38?
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.
it just works regardless of it.
It even works without the line.
would you prefer to remove it?
} | ||
} | ||
|
||
showUsage() | ||
showVersion() | ||
validateInputParams() | ||
|
||
process GENERATE_INPUT_VCF { |
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.
no need to process for both hg19/hg38?
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.
it just works regardless of that.
nextflow.config
Outdated
@@ -1,4 +1,7 @@ | |||
params { | |||
input_vcf = null | |||
input_variant = null | |||
|
|||
run_id = 1 | |||
ref_ver = "hg19" | |||
ref_dir = " " |
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.
better to do null
instead of " "
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.
I will apply that
we can't see the error message but i am pretty sure that's related with the docker version problem. |
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.
LGTM
No description provided.