Skip to content

TSPS-715 Add LowPassImputationQC wdl#1801

Open
mmorgantaylor wants to merge 73 commits intodevelopfrom
TSPS-715_glimpse_qc_wdl
Open

TSPS-715 Add LowPassImputationQC wdl#1801
mmorgantaylor wants to merge 73 commits intodevelopfrom
TSPS-715_glimpse_qc_wdl

Conversation

@mmorgantaylor
Copy link
Copy Markdown
Member

@mmorgantaylor mmorgantaylor commented Mar 20, 2026

Description

Here we add the InputQC wdl for the low pass imputation (Glimpse2) wdl, along with tests. Each failure mode has a test case.

Note: since this is a new wdl and a new set of tests, we expect the tests on this PR to fail, since there is no truth data yet. Once this PR is approved and merged, we will run the tests to generate truth data.

Crams that pass QC:

{ "qc_messages": "", "passes_qc": "true" }

Manifest that passes QC:

{ "qc_messages": "", "passes_qc": "true" }

No data provided:

{ "qc_messages": "No input data provided. Please provide either CRAM files or a CRAM manifest.", "passes_qc": "false" }

Both cram arrays and cram manifest provided:

{ "qc_messages": "Multiple input data types provided. Please provide only CRAM files (with corresponding CRAM index files and sample IDs) or a CRAM manifest.", "passes_qc": "false" }

Cram arrays provided, no crais:

{ "qc_messages": "CRAM indices and sample IDs are required when CRAM files are provided. Please provide both CRAM index files and a corresponding list of sample IDs.", "passes_qc": "false" }

Cram arrays provided, no sample ids:

{ "qc_messages": "CRAM indices and sample IDs are required when CRAM files are provided. Please provide both CRAM index files and a corresponding list of sample IDs.", "passes_qc": "false" }

Cram arrays provided, mismatched lengths:

{ "qc_messages": "Found different numbers of CRAMs (2), CRAIs (2), and sample IDs (1).", "passes_qc": "false" }

Cram arrays provided, duplicate sample_ids and duplicate cram_paths:

{ "qc_messages": "Duplicate sample IDs found: sample0, sample1\nDuplicate CRAM paths found: gs://fc-cddd72b5-323c-495c-9557-5057fff0275a/morgan_test/fakeCram0.cram, gs://fc-cddd72b5-323c-495c-9557-5057fff0275a/morgan_test/fakeCram1.cram", "passes_qc": "false" }

Cram arrays provided, duplicate cram paths:

{ "qc_messages": "Duplicate CRAM paths found: gs://fc-cddd72b5-323c-495c-9557-5057fff0275a/morgan_test/fakeCram0.cram", "passes_qc": "false" }

Cram arrays provided, file size too large:

{ "qc_messages": "The following CRAM files exceed the maximum allowed file size of 10GB: gs://pd-test-storage-public/CramToUnmappedBams/input/scientific/G94982.NA12878.WGS/NA12878.cram (19GB)", "passes_qc": "false" }

Manifest provided, duplicate sample_ids (basically just tests that a manifest gets properly converted to arrays):

{ "qc_messages": "Duplicate sample IDs found: sample0", "passes_qc": "false" }

Give your PR a concise yet descriptive title.
Please explain the changes you made here.
Explain the motivation for making this change. What existing problem does the pull request solve?
Mention any issues fixed, addressed, or otherwise related to this pull request, including issue numbers or hard links for issues in other repos.
You can delete these instructions once you have written your PR description.


Checklist

If you can answer "yes" to the following items, please add a checkmark next to the appropriate checklist item(s) and notify our WARP team by tagging @broadinstitute/warp-admins in a comment on this PR.

  • Did you add inputs, outputs, or tasks to a workflow?
  • Did you modify, delete or move: file paths, file names, input names, output names, or task names?
  • If you made a changelog update, did you update the pipeline version number?

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
Glimpse2LowPassImputation.changelog.md has not been changed and needs to be updated
Some changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
Glimpse2LowPassImputation.wdl has not had its version updated
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
Glimpse2LowPassImputation.wdl has not had its version updated
Glimpse2LowPassImputation.wdl and Glimpse2LowPassImputation.changelog.md do not have matching versions
Some WDLs or changelog files need updating. See output for details.
validation_failed

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🔍Version Validation Results:

Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

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.

4 participants