Skip to content

feat: gzip picard validate STDOUT#298

Merged
a-frantz merged 5 commits intomainfrom
validate_bam
Feb 16, 2026
Merged

feat: gzip picard validate STDOUT#298
a-frantz merged 5 commits intomainfrom
validate_bam

Conversation

@a-frantz
Copy link
Member

@a-frantz a-frantz commented Feb 15, 2026

Describe the problem or feature in addition to a link to the issues.

the non summary mode of validate_bam can output multiple lines per read in the input BAM, using more disk than the input file. This passing that output through gzip.

This also removes the x2 factor on disk size allocation for this task. We may instead want to keep that x2 factor but only if we're in full report mode?

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 previously approved these changes Feb 16, 2026
Co-authored-by: Andrew Thrasher <adthrasher@gmail.com>
@a-frantz a-frantz merged commit 192ed21 into main Feb 16, 2026
13 of 19 checks passed
@a-frantz a-frantz deleted the validate_bam branch February 16, 2026 14:27
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