Skip to content
New issue

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

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

Already on GitHub? # to your account

Update WDLs for upcoming pVACtools 4.0.0 release #93

Merged
merged 8 commits into from
Apr 24, 2023

Conversation

susannasiebert
Copy link
Member

No description provided.

}

Int space_needed_gb = 10 + round(size([input_fusions_zip], "GB") * 3)
runtime {
preemptible: 1
maxRetries: 2
docker: "griffithlab/pvactools:3.1.0"
docker: "susannakiwala/pvactools:4.0.0_rc_pvacview_v3"
Copy link
Member

Choose a reason for hiding this comment

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

Just commenting to note that we'll want to replace this with the final release version once it's ready.

~{if defined(iedb_retries) then "-r ~{iedb_retries}" else ""} \
~{if keep_tmp_files then "-k" else ""} \
~{if defined(normal_sample_name) then "--normal-sample-name ~{normal_sample_name}" else ""} \
~{if defined(net_chop_method) then "--net-chop-method ~{net_chop_method}" else ""} \
~{if netmhc_stab then "--netmhc-stab" else ""} \
~{if run_reference_proteome_similarity then "--run-reference-proteome-similarity" else ""} \
~{if defined(peptide_fasta) then "--peptide-fasta ~{peptide_fasta}" else ""} \
~{if defined(top_score_metric) then "-m ~{top_score_metric}" else ""} \
~{if defined(net_chop_threshold) then "--net-chop-threshold ~{net_chop_threshold}" else ""} \
~{if defined(additional_report_columns) then "-m ~{additional_report_columns}" else ""} \
Copy link
Member

Choose a reason for hiding this comment

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

It looks like -m is here twice?

@malachig malachig assigned malachig and unassigned malachig Apr 24, 2023
@malachig
Copy link
Member

This all seems to be working as expected in my test with real datasets.

@susannasiebert susannasiebert marked this pull request as ready for review April 24, 2023 22:08
@susannasiebert susannasiebert merged commit 6ec4ed3 into wustl-oncology:main Apr 24, 2023
Copy link
Member

@malachig malachig left a comment

Choose a reason for hiding this comment

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

This looks good to me and seems to be working as expected.

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

Successfully merging this pull request may close these issues.

3 participants