Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CrosscheckFP WDL #60

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

CrosscheckFP WDL #60

wants to merge 9 commits into from

Conversation

yueyaog
Copy link
Collaborator

@yueyaog yueyaog commented Jul 18, 2024

@MicahR-Y Here is the original crosscheck FP WDL. Feel free to suggest changes or send me the updated version you have.

@MicahR-Y MicahR-Y self-requested a review July 30, 2024 17:27
Copy link
Collaborator

@MicahR-Y MicahR-Y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yueyaog I made some suggestions for this code. The two major points are changing the inputs to an array to allow for more than 2 samples to be cross-checked at once and including an option for not failing the workflow if the samples don't match.

String output_name
String picard_docker = "us.gcr.io/broad-gotc-prod/genomes-in-the-cloud:2.4.1-1540490856"
}

Copy link
Collaborator

@MicahR-Y MicahR-Y Jul 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Int mismatch_rc = if fail_on_mismatch then 1 else 2


workflow CrosscheckFingerprintsWorkflow {
input {
File input_bamA
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
File input_bamA

workflow CrosscheckFingerprintsWorkflow {
input {
File input_bamA
File input_bamB
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
File input_bamB
Array[File] input_bams

File input_bamB
File hapmap
File reference_fasta
String output_name
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
String output_name
String output_name
Boolean fail_on_mismatch = false


call CrosscheckFingerprints {
input:
input_bamA = input_bamA,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
input_bamA = input_bamA,

I=~{input_bamB} \
HAPLOTYPE_MAP=~{hapmap} \
CROSSCHECK_BY=SAMPLE \
O=~{output_name}.crosscheck_metrics \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
O=~{output_name}.crosscheck_metrics \
O=~{output_name}.crosscheck_metrics.txt \

HAPLOTYPE_MAP=~{hapmap} \
CROSSCHECK_BY=SAMPLE \
O=~{output_name}.crosscheck_metrics \
REFERENCE_SEQUENCE=~{reference} \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
REFERENCE_SEQUENCE=~{reference} \
REFERENCE_SEQUENCE=~{reference} \
EXIT_CODE_WHEN_MISMATCH=~{mismatch_rc} \
~{extra_args} \

}

output {
File crosscheckmetrics = "~{output_name}.crosscheck_metrics"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
File crosscheckmetrics = "~{output_name}.crosscheck_metrics"
File crosscheckmetrics = "~{output_name}.crosscheck_metrics.txt"
String crosscheck_results = if read_int("rc") == 0 then "PASS" else "FAIL"

File crosscheckmetrics = "~{output_name}.crosscheck_metrics"
}
}
#
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove the commented out task here?

output_name = output_name,
picard_docker = picard_docker
}
# call FingerprintsResult {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you remove the commented out call here?


runtime {
docker: picard_docker
disks: "local-disk " + disk_gb + " HDD"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
disks: "local-disk " + disk_gb + " HDD"
disks: "local-disk " + disk_gb + " HDD"
continueOnReturnCode: [0, 2]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants