Skip to content

Commit 668d763

Browse files
committed
Addressed PR comments.
1 parent 1d90cc2 commit 668d763

4 files changed

+3
-22
lines changed

scripts/cnv_wdl/cnv_common_tasks.wdl

-14
Original file line numberDiff line numberDiff line change
@@ -242,11 +242,6 @@ task ScatterIntervals {
242242
}
243243

244244
task PostprocessGermlineCNVCalls {
245-
Array[File] calling_configs
246-
Array[File] denoising_configs
247-
Array[File] gcnvkernel_version
248-
Array[File] sharded_interval_lists
249-
250245
String entity_id
251246
Array[File] gcnv_calls_tars
252247
Array[File] gcnv_model_tars
@@ -278,21 +273,12 @@ task PostprocessGermlineCNVCalls {
278273
export GATK_LOCAL_JAR=${default="/root/gatk.jar" gatk4_jar_override}
279274

280275
# untar calls to CALLS_0, CALLS_1, etc directories and build the command line
281-
# also copy over shard config and interval files
282276
gcnv_calls_tar_array=(${sep=" " gcnv_calls_tars})
283-
calling_configs_array=(${sep=" " calling_configs})
284-
denoising_configs_array=(${sep=" " denoising_configs})
285-
gcnvkernel_version_array=(${sep=" " gcnvkernel_version})
286-
sharded_interval_lists_array=(${sep=" " sharded_interval_lists})
287277
calls_args=""
288278
for index in ${dollar}{!gcnv_calls_tar_array[@]}; do
289279
gcnv_calls_tar=${dollar}{gcnv_calls_tar_array[$index]}
290280
mkdir CALLS_$index
291281
tar xzf $gcnv_calls_tar -C CALLS_$index
292-
cp ${dollar}{calling_configs_array[$index]} CALLS_$index/
293-
cp ${dollar}{denoising_configs_array[$index]} CALLS_$index/
294-
cp ${dollar}{gcnvkernel_version_array[$index]} CALLS_$index/
295-
cp ${dollar}{sharded_interval_lists_array[$index]} CALLS_$index/
296282
calls_args="$calls_args --calls-shard-path CALLS_$index"
297283
done
298284

scripts/cnv_wdl/germline/cnv_germline_case_scattered_workflow.wdl

+3
Original file line numberDiff line numberDiff line change
@@ -179,6 +179,9 @@ workflow CNVGermlineCaseScatteredWorkflow {
179179
}
180180

181181
output {
182+
Array[File] preprocessed_intervals = CNVGermlineCaseWorkflow.preprocessed_intervals
183+
Array[Array[File]] read_counts_entity_id = CNVGermlineCaseWorkflow.read_counts_entity_id
184+
Array[Array[File]] read_counts = CNVGermlineCaseWorkflow.read_counts
182185
Array[File] contig_ploidy_calls_tars = CNVGermlineCaseWorkflow.contig_ploidy_calls_tar
183186
Array[Array[File]] gcnv_calls_tars = CNVGermlineCaseWorkflow.gcnv_calls_tars
184187
Array[Array[File]] gcnv_tracking_tars = CNVGermlineCaseWorkflow.gcnv_tracking_tars

scripts/cnv_wdl/germline/cnv_germline_case_workflow.wdl

-4
Original file line numberDiff line numberDiff line change
@@ -214,10 +214,6 @@ workflow CNVGermlineCaseWorkflow {
214214
scatter (sample_index in range(length(normal_bams))) {
215215
call CNVTasks.PostprocessGermlineCNVCalls {
216216
input:
217-
calling_configs = GermlineCNVCallerCaseMode.calling_config_json,
218-
denoising_configs = GermlineCNVCallerCaseMode.denoising_config_json,
219-
gcnvkernel_version = GermlineCNVCallerCaseMode.gcnvkernel_version_json,
220-
sharded_interval_lists = GermlineCNVCallerCaseMode.sharded_interval_list,
221217
entity_id = CollectCounts.entity_id[sample_index],
222218
gcnv_calls_tars = GermlineCNVCallerCaseMode.gcnv_calls_tar,
223219
gcnv_model_tars = gcnv_model_tars,

scripts/cnv_wdl/germline/cnv_germline_cohort_workflow.wdl

-4
Original file line numberDiff line numberDiff line change
@@ -262,10 +262,6 @@ workflow CNVGermlineCohortWorkflow {
262262
scatter (sample_index in range(length(CollectCounts.entity_id))) {
263263
call CNVTasks.PostprocessGermlineCNVCalls {
264264
input:
265-
calling_configs = GermlineCNVCallerCohortMode.calling_config_json,
266-
denoising_configs = GermlineCNVCallerCohortMode.denoising_config_json,
267-
gcnvkernel_version = GermlineCNVCallerCohortMode.gcnvkernel_version_json,
268-
sharded_interval_lists = GermlineCNVCallerCohortMode.sharded_interval_list,
269265
entity_id = CollectCounts.entity_id[sample_index],
270266
gcnv_calls_tars = GermlineCNVCallerCohortMode.gcnv_calls_tar,
271267
gcnv_model_tars = GermlineCNVCallerCohortMode.gcnv_model_tar,

0 commit comments

Comments
 (0)