Skip to content

Commit f251675

Browse files
authored
Merge pull request #650 from broadinstitute/bump-viral-ngs
bump viral-ngs 3.0.10, remove GATK IndelRealigner deps, update lofreq pipeline
2 parents eec1073 + 6c3555f commit f251675

16 files changed

+143
-127
lines changed

.github/workflows/build.yml

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@ jobs:
4848
runs-on: ubuntu-24.04
4949
steps:
5050
- name: checkout repository
51-
uses: actions/checkout@v4
51+
uses: actions/checkout@v6
5252
# fetch git tags (tagged releases) because
53-
# actions/checkout@v4 does either a full checkout or a shallow checkout without tags
53+
# actions/checkout@v6 does either a full checkout or a shallow checkout without tags
5454
- name: fetch tags
5555
run: git fetch --prune --unshallow --tags
5656
- name: Programmatic environment setup
@@ -80,7 +80,7 @@ jobs:
8080
run: |
8181
env
8282
- name: install python
83-
uses: actions/setup-python@v5
83+
uses: actions/setup-python@v6
8484
with:
8585
python-version: '3.10'
8686
- name: install system dependencies
@@ -103,9 +103,9 @@ jobs:
103103
runs-on: ubuntu-24.04
104104
steps:
105105
- name: checkout repository
106-
uses: actions/checkout@v4
106+
uses: actions/checkout@v6
107107
# fetch git tags (tagged releases) because
108-
# actions/checkout@v4 does either a full checkout or a shallow checkout without tags
108+
# actions/checkout@v6 does either a full checkout or a shallow checkout without tags
109109
- name: fetch tags
110110
run: git fetch --prune --unshallow --tags
111111
- name: Programmatic environment setup
@@ -131,7 +131,7 @@ jobs:
131131
echo "GITHUB_ACTIONS_BRANCH=$GITHUB_ACTIONS_BRANCH"
132132
echo "GITHUB_ACTIONS_BRANCH=$GITHUB_ACTIONS_BRANCH" >> $GITHUB_ENV
133133
- name: install java
134-
uses: actions/setup-java@v4
134+
uses: actions/setup-java@v5
135135
with:
136136
distribution: 'temurin'
137137
java-version: '17'
@@ -155,9 +155,9 @@ jobs:
155155
runs-on: ubuntu-24.04
156156
steps:
157157
- name: checkout repository
158-
uses: actions/checkout@v4
158+
uses: actions/checkout@v6
159159
# fetch git tags (tagged releases) because
160-
# actions/checkout@v4 does either a full checkout or a shallow checkout without tags
160+
# actions/checkout@v6 does either a full checkout or a shallow checkout without tags
161161
- name: fetch tags
162162
run: git fetch --prune --unshallow --tags
163163
- name: Programmatic environment setup
@@ -183,7 +183,7 @@ jobs:
183183
echo "GITHUB_ACTIONS_BRANCH=$GITHUB_ACTIONS_BRANCH"
184184
echo "GITHUB_ACTIONS_BRANCH=$GITHUB_ACTIONS_BRANCH" >> $GITHUB_ENV
185185
- name: install python
186-
uses: actions/setup-python@v5
186+
uses: actions/setup-python@v6
187187
with:
188188
python-version: '3.10'
189189
- name: install docs dependencies
@@ -202,9 +202,9 @@ jobs:
202202
runs-on: ubuntu-24.04
203203
steps:
204204
- name: checkout repository
205-
uses: actions/checkout@v4
205+
uses: actions/checkout@v6
206206
# fetch git tags (tagged releases) because
207-
# actions/checkout@v4 does either a full checkout or a shallow checkout without tags
207+
# actions/checkout@v6 does either a full checkout or a shallow checkout without tags
208208
- name: fetch tags
209209
run: git fetch --prune --unshallow --tags
210210
- name: Programmatic environment setup
@@ -237,7 +237,7 @@ jobs:
237237
sudo rm -rf "/usr/local/share/boost"
238238
sudo rm -rf "$AGENT_TOOLSDIRECTORY"
239239
- name: install java
240-
uses: actions/setup-java@v4
240+
uses: actions/setup-java@v5
241241
with:
242242
distribution: 'temurin'
243243
java-version: '17'
@@ -259,9 +259,9 @@ jobs:
259259
runs-on: ubuntu-24.04
260260
steps:
261261
- name: checkout repository
262-
uses: actions/checkout@v4
262+
uses: actions/checkout@v6
263263
# fetch git tags (tagged releases) because
264-
# actions/checkout@v4 does either a full checkout or a shallow checkout without tags
264+
# actions/checkout@v6 does either a full checkout or a shallow checkout without tags
265265
- name: fetch tags
266266
run: git fetch --prune --unshallow --tags
267267
- name: Programmatic environment setup
@@ -294,7 +294,7 @@ jobs:
294294
sudo rm -rf "/usr/local/share/boost"
295295
sudo rm -rf "$AGENT_TOOLSDIRECTORY"
296296
- name: install python
297-
uses: actions/setup-python@v5
297+
uses: actions/setup-python@v6
298298
with:
299299
python-version: '3.10'
300300
- name: install system dependencies
@@ -305,7 +305,7 @@ jobs:
305305
run: |
306306
pip3 install miniwdl docker[tls] six
307307
- name: Set up Docker Buildx
308-
uses: docker/setup-buildx-action@v3
308+
uses: docker/setup-buildx-action@v4
309309
- name: test with miniwdl
310310
shell: bash
311311
run: |
@@ -330,9 +330,9 @@ jobs:
330330
DX_PROJECT: project-F8PQ6380xf5bK0Qk0YPjB17P
331331
steps:
332332
- name: checkout repository
333-
uses: actions/checkout@v4
333+
uses: actions/checkout@v6
334334
# fetch git tags (tagged releases) because
335-
# actions/checkout@v4 does either a full checkout or a shallow checkout without tags
335+
# actions/checkout@v6 does either a full checkout or a shallow checkout without tags
336336
- name: fetch tags
337337
run: git fetch --prune --unshallow --tags
338338
- name: Programmatic environment setup
@@ -363,11 +363,11 @@ jobs:
363363
echo "${{ github.event.action }}"
364364
echo "${{ github.event.pull_request.merged }}"
365365
- name: install python
366-
uses: actions/setup-python@v5
366+
uses: actions/setup-python@v6
367367
with:
368368
python-version: '3.10'
369369
- name: install java
370-
uses: actions/setup-java@v4
370+
uses: actions/setup-java@v5
371371
with:
372372
distribution: 'temurin'
373373
java-version: '17'

pipes/WDL/tasks/tasks_assembly.wdl

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ task assemble {
1616

1717
Int? machine_mem_gb
1818
Int? cpu
19-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-assemble"
19+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-assemble"
2020
}
2121
parameter_meta{
2222
reads_unmapped_bam: {
@@ -124,7 +124,7 @@ task select_references {
124124
Int? skani_c
125125
Int? skani_n
126126

127-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-assemble"
127+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-assemble"
128128
Int machine_mem_gb = 4
129129
Int cpu = 2
130130
Int disk_size = 100
@@ -223,7 +223,7 @@ task scaffold {
223223
Float? scaffold_min_pct_contig_aligned
224224

225225
Int? machine_mem_gb
226-
String docker="quay.io/broadinstitute/viral-ngs:3.0.9-assemble"
226+
String docker="quay.io/broadinstitute/viral-ngs:3.0.10-assemble"
227227

228228
# do this in multiple steps in case the input doesn't actually have "assembly1-x" in the name
229229
String sample_name = basename(basename(contigs_fasta, ".fasta"), ".assembly1-spades")
@@ -475,7 +475,7 @@ task skani_triangle {
475475
Int compression_factor = 10
476476
Int min_aligned_frac = 15
477477

478-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-assemble"
478+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-assemble"
479479
Int machine_mem_gb = 8
480480
Int cpu = 4
481481
Int disk_size = 100
@@ -636,7 +636,7 @@ task ivar_trim_stats {
636636
String out_basename = "ivar_trim_stats"
637637
String flowcell = ""
638638

639-
String docker = "quay.io/broadinstitute/py3-bio:0.1.4"
639+
String docker = "quay.io/broadinstitute/py3-bio:0.1.5"
640640
}
641641
parameter_meta {
642642
ivar_trim_stats_tsv: {
@@ -715,7 +715,7 @@ task align_reads {
715715
716716
Int? cpu
717717
Int? machine_mem_gb
718-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-core"
718+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-core"
719719

720720
String sample_name = basename(basename(basename(reads_unmapped_bam, ".bam"), ".taxfilt"), ".clean")
721721
}
@@ -724,10 +724,6 @@ task align_reads {
724724
# 7x input BAM to account for bbnorm preprocessing temp file
725725
Int disk_size = ceil((7 * size(reads_unmapped_bam, "GB") + 2 * size(reference_fasta, "GB") + 100) / 750.0) * 750
726726

727-
# Skip indel realignment for large BAMs (>1GB) to save runtime
728-
Float reads_bam_size_gb = size(reads_unmapped_bam, "GB")
729-
Boolean skip_realign = reads_bam_size_gb >= 1.0
730-
731727
# bbnorm preprocessing: max output reads is half the downsample threshold
732728
Int bbnorm_max_output_reads = read_count_downsample_threshold / 2
733729

@@ -808,7 +804,6 @@ task align_reads {
808804
--aligner ~{aligner} \
809805
~{'--aligner_options "' + aligner_options + '"'} \
810806
~{true='--skipMarkDupes' false="" skip_mark_dupes} \
811-
~{true='--skipRealign' false="" skip_realign} \
812807
--JVMmemory "$mem_in_mb"m \
813808
~{"--NOVOALIGN_LICENSE_PATH=" + novocraft_license} \
814809
--loglevel=DEBUG
@@ -883,7 +878,7 @@ task refine_assembly_with_aligned_reads {
883878
Int min_coverage = 3
884879

885880
Int machine_mem_gb = 8
886-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-assemble"
881+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-assemble"
887882
}
888883

889884
Int disk_size = 375
@@ -1019,7 +1014,7 @@ task run_discordance {
10191014
String out_basename = "run"
10201015
Int min_coverage = 4
10211016

1022-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-core"
1017+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-core"
10231018
}
10241019
parameter_meta {
10251020
reads_aligned_bam: {
@@ -1266,7 +1261,7 @@ task wgsim {
12661261
Int? random_seed
12671262

12681263
Int machine_mem_gb = 7
1269-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-assemble"
1264+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-assemble"
12701265
}
12711266

12721267
parameter_meta {

pipes/WDL/tasks/tasks_demux.wdl

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ task merge_tarballs {
66
String out_filename
77

88
Int? machine_mem_gb
9-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-core"
9+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-core"
1010
}
1111

1212
Int disk_size = 2625
@@ -97,7 +97,7 @@ task revcomp_i5 {
9797
input {
9898
File old_sheet
9999
Boolean revcomp=true
100-
String docker = "quay.io/broadinstitute/py3-bio:0.1.4"
100+
String docker = "quay.io/broadinstitute/py3-bio:0.1.5"
101101
}
102102
String new_base = basename(basename(old_sheet, '.txt'), '.tsv')
103103
Int disk_size = 50
@@ -179,7 +179,7 @@ task illumina_demux {
179179
Int? machine_mem_gb
180180
# Note: GCP local SSDs must be allocated in pairs (2, 4, 8, 16, 24 × 375GB), so use 3000 (8 SSDs) instead of 2625 (7 SSDs)
181181
Int disk_size = 3000
182-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-core"
182+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-core"
183183
}
184184

185185
parameter_meta {
@@ -817,7 +817,7 @@ task get_illumina_run_metadata {
817817
String? sequencing_center
818818

819819
Int? machine_mem_gb
820-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-core"
820+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-core"
821821
}
822822

823823
parameter_meta {
@@ -920,7 +920,7 @@ task demux_fastqs {
920920
Int? machine_mem_gb
921921
Int max_cpu = 32 # Maximum CPU cap for autoscaling (use 16 for 2-barcode, 64 for 3-barcode)
922922
Int disk_size = 750
923-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-core"
923+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-core"
924924
}
925925

926926
# Calculate total input size for autoscaling
@@ -1048,7 +1048,7 @@ task merge_demux_metrics {
10481048
input {
10491049
Array[File]+ metrics_files
10501050
String output_filename = "merged_demux_metrics.txt"
1051-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-core"
1051+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-core"
10521052
}
10531053

10541054
parameter_meta {

pipes/WDL/tasks/tasks_interhost.wdl

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ task multi_align_mafft_ref {
160160
Float? mafft_gapOpeningPenalty
161161

162162
Int? machine_mem_gb
163-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-phylo"
163+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-phylo"
164164
}
165165

166166
String fasta_basename = basename(reference_fasta, '.fasta')
@@ -206,7 +206,7 @@ task multi_align_mafft {
206206
Float? mafft_gapOpeningPenalty
207207

208208
Int? machine_mem_gb
209-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-phylo"
209+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-phylo"
210210
}
211211

212212
Int disk_size = 200
@@ -348,7 +348,7 @@ task index_ref {
348348
File? novocraft_license
349349

350350
Int? machine_mem_gb
351-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-core"
351+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-core"
352352
}
353353

354354
Int disk_size = 100
@@ -470,7 +470,7 @@ task merge_vcfs_gatk {
470470
File ref_fasta
471471

472472
Int? machine_mem_gb
473-
String docker = "quay.io/broadinstitute/viral-ngs:3.0.9-phylo"
473+
String docker = "quay.io/broadinstitute/viral-ngs:3.0.10-phylo"
474474

475475
String output_prefix = "merged"
476476
}

0 commit comments

Comments
 (0)