Skip to content
This repository was archived by the owner on Jan 27, 2020. It is now read-only.

Commit 436cf23

Browse files
author
Szilveszter Juhos
authored
Merge pull request #678 from szilvajuhos/master
Changing VEP to v92 and adjusting CPUs
2 parents e100490 + 2c1f170 commit 436cf23

6 files changed

Lines changed: 9 additions & 7 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
1414

1515
### `Changed`
1616

17+
- [#678](https://github.com/SciLifeLab/Sarek/pull/678) - Changing VEP to v92 and adjusting CPUs for VEP
1718
- [#663](https://github.com/SciLifeLab/Sarek/pull/663) - Update `do_release.sh` script
1819
- [#671](https://github.com/SciLifeLab/Sarek/pull/671) - publishDir modes are now params
1920

annotate.nf

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,13 +215,14 @@ process RunVEP {
215215
script:
216216
finalannotator = annotator == "snpeff" ? 'merge' : 'vep'
217217
genome = params.genome == 'smallGRCh37' ? 'GRCh37' : params.genome
218+
cache_version = params.genome == 'GRCh38' ? 92 : 91
218219
"""
219220
/opt/vep/src/ensembl-vep/vep --dir /opt/vep/.vep/ \
220221
-i ${vcf} \
221222
-o ${vcf.simpleName}_VEP.ann.vcf \
222223
--assembly ${genome} \
223224
--cache \
224-
--cache_version 91 \
225+
--cache_version ${cache_version} \
225226
--database \
226227
--everything \
227228
--filter_common \

conf/uppmax-localhost.config

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,15 +128,16 @@ process {
128128
memory = {params.totalMemory}
129129
}
130130
withName:RunSnpeff {
131+
cpus = 1
131132
errorStrategy = {task.exitStatus == 143 ? 'retry' : 'ignore'}
132-
memory = {params.totalMemory} // TODO Does SnpEff really require that much?
133+
memory = {params.singleCPUMem * task.attempt}
133134
}
134135
withName:RunStrelka {
135136
cpus = 16
136137
memory = {params.totalMemory}
137138
}
138139
withName:RunVEP {
140+
cpus = 16
139141
errorStrategy = {task.exitStatus == 143 ? 'retry' : 'ignore'}
140-
memory = {params.totalMemory} // TODO Does VEP really require that much?
141142
}
142143
}

conf/uppmax-slurm.config

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,6 @@ process {
129129
time = {params.runTime * task.attempt}
130130
}
131131
withName:RunVEP {
132-
cpus = 1
133132
errorStrategy = { task.exitStatus == 143 ? 'retry' : 'ignore' }
134133
}
135134
}

containers/vepgrch38/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ LABEL \
88
# Setup ENV variables
99
ENV \
1010
GENOME=GRCh38 \
11-
VEP_VERSION=91
11+
VEP_VERSION=92
1212

1313
# Download Genome
1414
RUN \

docs/CONTAINERS.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,8 @@ We provide script to build/push or pull all containers
120120

121121
### vepgrch38 [![vepgrch38-docker status][vepgrch38-docker-badge]][vepgrch38-docker-link]
122122

123-
- Based on `willmclaren/ensembl-vep:release_90.6`
124-
- Contain **[VEP][vep-link]** 90.5
123+
- Based on `willmclaren/ensembl-vep:release_92`
124+
- Contain **[VEP][vep-link]** 92
125125
- Contain GRCh38
126126

127127
[allelecount-link]: https://github.com/cancerit/alleleCount

0 commit comments

Comments
 (0)