Skip to content

Commit

Permalink
allow additional options for picard in FastqToUBAM (#518)
Browse files Browse the repository at this point in the history
* allow additional options for picard in FastqToUBAM

via a new additional_picard_option_string input that's passed to picard directly if set

* s/additional_picard_option_string/additional_picard_options/g

rename input to additional_picard_options to be more consistent with the naming of inputs in other tasks where we allow the user to pass in arbitrary arg strings
  • Loading branch information
tomkinsc authored Feb 23, 2024
1 parent 7748df1 commit 370ec84
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion pipes/WDL/tasks/tasks_read_utils.wdl
Original file line number Diff line number Diff line change
Expand Up @@ -365,6 +365,7 @@ task FastqToUBAM {
String? run_date
String platform_name
String? sequencing_center
String? additional_picard_options
String docker = "quay.io/broadinstitute/viral-core:2.2.4"
}
Expand All @@ -375,6 +376,7 @@ task FastqToUBAM {
sample_name: { description: "Sample name. This is required and will populate the 'SM' read group value and will be used as the output filename (must be filename-friendly)." }
library_name: { description: "Library name. This is required and will populate the 'LB' read group value. SM & LB combinations must be identical for any sequencing reads generated from the same sequencing library, and must be distinct for any reads generated from different libraries." }
platform_name: { description: "Sequencing platform. This is required and will populate the 'PL' read group value. Must be one of CAPILLARY, DNBSEQ, HELICOS, ILLUMINA, IONTORRENT, LS454, ONT, PACBIO, or SOLID." }
additional_picard_options: { description: "A string containing additional options to pass to picard FastqToSam beyond those made explicitly available as inputs to this task. For valid values, see: https://broadinstitute.github.io/picard/command-line-overview.html#FastqToSam" }
}
command {
set -ex -o pipefail
Expand All @@ -395,7 +397,7 @@ task FastqToUBAM {
${"PLATFORM_UNIT=" + platform_unit} \
${"RUN_DATE=" + run_date} \
${"PLATFORM=" + platform_name} \
${"SEQUENCING_CENTER=" + sequencing_center}
${"SEQUENCING_CENTER=" + sequencing_center} ${additional_picard_options}
}
runtime {
docker: docker
Expand Down

0 comments on commit 370ec84

Please sign in to comment.