Skip to content

Conversation

@bagnacan
Copy link
Contributor

@bagnacan bagnacan commented Feb 8, 2026

This PR fixes a bug in the paraphase module.

Paraphase's output vcf/vcf-index files are created only if the provided input BAM file has reads in the analyzed region. When the analyzed region has no reads, paraphase creates no vcf/vcf-index files. For this reason, the paraphase module should zip these files only when they are available.
This PR introduces a check for the availability of these files, as well as a new test to verify that these outputs are indeed optional.
Test data needed to be extended to cover for this corner case (see test-dataset PR here).

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If necessary, include test data in your PR.

@bagnacan bagnacan marked this pull request as draft February 8, 2026 11:40
@bagnacan bagnacan marked this pull request as ready for review February 8, 2026 13:27
@bagnacan bagnacan force-pushed the paraphase-optional-output branch from ac5d305 to b34f72d Compare February 8, 2026 13:30
@bagnacan bagnacan force-pushed the paraphase-optional-output branch from b34f72d to 1acdf8d Compare February 8, 2026 13:32
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