Skip to content

Commit

Permalink
Clarify TODO comment about fragment lengths
Browse files Browse the repository at this point in the history
  • Loading branch information
vincent-czi committed Jul 24, 2023
1 parent 40b111d commit abc0884
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion workflows/short-read-mngs/host_filter.wdl
Original file line number Diff line number Diff line change
Expand Up @@ -428,7 +428,10 @@ task kallisto {
Int cpu = 16
}
Boolean paired = defined(fastp2_fastq)
# TODO: input fragment length parameters for non-paired-end (l = average, s = std dev)
# TODO: It would be better to have fragment length params for non-paired-end
# (l = average, s = std dev) get set dynamically based on the input data.
# See this GitHub comment for more background info:
# https://github.com/chanzuckerberg/czid-workflows/pull/282#issuecomment-1647494061
String kallisto_invocation = "/kallisto/kallisto quant"
+ " -i '${kallisto_idx}' -o $(pwd) --plaintext ${if (paired) then '' else '--single -l 200 -s 20'} ${kallisto_options}"
+ " '~{fastp1_fastq}'" + if (defined(fastp2_fastq)) then " '~{fastp2_fastq}'" else ""
Expand Down

0 comments on commit abc0884

Please sign in to comment.