-
Notifications
You must be signed in to change notification settings - Fork 7
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
Module/gatk rnaseq/1.0 #184
base: master
Are you sure you want to change the base?
Changes from 12 commits
e8874a5
c1f7318
f0310f4
b760336
e2b4623
83342ab
930fb25
cc95c65
12d743f
361c0b7
d15b7d5
1a11e3f
dc4620f
4abb545
f422933
4acda94
f697c8f
c299580
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,72 @@ | ||
lcr-modules: | ||
|
||
gatk_rnaseq: | ||
|
||
inputs: | ||
# Available wildcards: {seq_type} {genome_build} {sample_id} | ||
sample_bam: "__UPDATE__" | ||
sample_bai: "__UPDATE__" | ||
|
||
|
||
scratch_subdirectories: [] | ||
|
||
options: | ||
java_opts: "-XX:ConcGCThreads=1" | ||
gatk_variant_calling: | ||
min_conf_thres: 20.0 | ||
gatk_variant_filtration: | ||
window: 35 # window size between SNPs in cluster | ||
cluster_size: 3 # at least 3 SNPs in cluster | ||
# hard filtering (filters OUT) based on metrics: | ||
# FS (FisherStrand): Phred-scale probability that there is a strand bias from a Fisher's test. (default FS > 30.0) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for detailed documentation! |
||
# QD (QualByDepth): variant confidence/unfiltered depth (normalizes variant quality to avoid inflation from deep coverage) (default QD < 2.0) | ||
# DP (depth): minimum depth (default DP < 5.0) | ||
filter_expression: "-filter-expression \"FS > 30.0\" -filter-name FS -filter-expression \"QD < 2.0\" -filter-name QD -filter-expression \"DP < 5.0\" -filter-name DP" | ||
gatk_rnaseq_filter_passed: | ||
params: "-f '.,PASS' " | ||
# Can be modified to filter on additional criteria using bcftools view syntax | ||
# For example, to remove all variants with -log10(POPAF) > 4.0: | ||
#"-f '.,PASS' -i 'INFO/POPAF > 4'" | ||
|
||
|
||
conda_envs: | ||
gatk_rnaseq: "{MODSDIR}/envs/gatk-4.1.8.1.yaml" | ||
bcftools: "{MODSDIR}/envs/bcftools-1.10.2.yaml" | ||
|
||
threads: | ||
gatk_splitntrim: 12 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is possible to combine these keys and reuse them across rules. In other words, if several rules need the same number of threads, they can refer to the same key in config. Same can be applied to resources as well. I think this reduces the number of keys to specify/adjust if needed, and reduces complexity of the config. What do you think? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's possible, but wouldn't it also cause confusion if there's ever a need to change the numbers? Also, would there be a unique name that could be applied to a subset of the rules ("thread_12" would be non-descriptive and wouldn't indicate which rules would use this parameter)? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You are right, let's keep the more detailed and informative names |
||
gatk_base_recalibration: 12 | ||
gatk_applybqsr: 12 | ||
gatk_variant_calling: 24 | ||
gatk_variant_filtration: 24 | ||
merge_vcfs: 10 | ||
gatk_rnaseq_passed: 10 | ||
gnomad_filter: 4 | ||
|
||
resources: | ||
gatk_splitntrim: | ||
mem_mb: 48000 | ||
gatk_base_recalibration: | ||
mem_mb: 12000 | ||
gatk_applybqsr: | ||
mem_mb: 12000 | ||
gatk_variant_calling: | ||
mem_mb: 48000 | ||
bam: 1 | ||
gatk_variant_filtration: | ||
mem_mb: 12000 | ||
merge_vcfs: | ||
mem_mb: 10000 | ||
gatk_rnaseq_passed: | ||
mem_mb: 10000 | ||
gnomad_filter: | ||
mem_mb: 2000 | ||
|
||
pairing_config: | ||
mrna: | ||
run_paired_tumours: False | ||
run_unpaired_tumours_with: "no_normal" | ||
run_paired_tumours_as_unpaired: True | ||
|
||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
../../../../envs/bcftools/bcftools-1.10.2.yaml |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
../../../../envs/gatk/gatk-4.1.8.1.yaml |
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.
This needs new line added at the end
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.
Done
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 still shows no new line here, maybe I am looking at outdated file?
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.
Sorry, I must have missed it. It's fixed