Skip to content
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

Merged
merged 5 commits into from
Dec 18, 2024
Merged

Conversation

jylee-bcm
Copy link
Contributor

No description provided.

@hyunhwan-jeong
Copy link

Can you find why the smoke test failed. @jylee-bcm

##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

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?

Copy link
Contributor Author

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 {

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?

Copy link
Contributor Author

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 = " "

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 " "

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will apply that

@jylee-bcm
Copy link
Contributor Author

Can you find why the smoke test failed. @jylee-bcm

we can't see the error message but i am pretty sure that's related with the docker version problem.

Copy link
Contributor

@hyunhwan-bcm hyunhwan-bcm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jylee-bcm jylee-bcm mentioned this pull request Dec 18, 2024
3 tasks
@jylee-bcm jylee-bcm merged commit 75f7ca1 into develop Dec 18, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants