Skip to content

feat: make FASTQ analysis optional#315

Open
adthrasher wants to merge 4 commits into
mainfrom
feat/qc_fastq_option
Open

feat: make FASTQ analysis optional#315
adthrasher wants to merge 4 commits into
mainfrom
feat/qc_fastq_option

Conversation

@adthrasher
Copy link
Copy Markdown
Member

@adthrasher adthrasher commented May 18, 2026

This makes the FASTQ portion of the QC pipeline optional. The default remains to run, but this allows a user to disable the analysis that depends on FASTQ files.

Also contains an off-topic change to add exceptions for EmptyOutputs to tasks with no expected outputs, since that rule was added to Sprocket.

Before submitting this PR, please make sure:

  • You have added a few sentences describing the PR here.
  • The code passes all CI tests without any errors or warnings.
  • You have added tests (when appropriate).
  • You have added an entry in any relevant CHANGELOGs (when appropriate).
  • If you have made any changes to the scripts/ or docker/ directories, please ensure any image versions have been incremented accordingly!
  • You have updated the README or other documentation to account for these changes (when appropriate).

@adthrasher adthrasher self-assigned this May 18, 2026
Comment thread workflows/qc/quality-check-standard.wdl Outdated
@adthrasher adthrasher requested a review from a-frantz May 18, 2026 16:02
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.

1 participant