Skip to content

Adding WILDS WDL modules for bowtie & bowtie2#260

Merged
tefirman merged 6 commits intomainfrom
add-bowtie
Mar 10, 2026
Merged

Adding WILDS WDL modules for bowtie & bowtie2#260
tefirman merged 6 commits intomainfrom
add-bowtie

Conversation

@tefirman
Copy link
Member

@tefirman tefirman commented Mar 8, 2026

Type of Change

  • New modules

Description

Adds two new alignment modules to the WILDS WDL library:

  • ww-bowtie - Short-read alignment using Bowtie (v1.3.1), an ultrafast, memory-efficient aligner optimized for reads up to ~50 bp. Includes bowtie_build and bowtie_align tasks.
  • ww-bowtie2 - Sequence alignment using Bowtie 2 (v2.5.4), a fast and sensitive gapped-read aligner supporting variable-length reads. Includes bowtie2_build and bowtie2_align tasks.

Both modules support paired-end and single-end alignment, produce sorted BAM + BAI output via samtools, and follow existing WILDS conventions (matching the ww-bwa module pattern).

Testing

How did you test these changes?
Ran testrun.wdl for both modules using test data from ww-testdata (reference genome + FASTQ files). Both paired-end and single-end samples aligned successfully.

What workflow engine did you use?
Sprocket (v0.18.0)

Did the tests pass?
Yes - both modules produce sorted BAM and BAI files for all test samples. Sprocket lint also passes clean.

Documentation

  • I updated the README (if applicable)
  • I added/updated parameter descriptions in the WDL (if applicable)
  • I ran make docs-preview to check documentation rendering (if applicable)

Additional Context

Both Docker images (getwilds/bowtie:1.3.1 and getwilds/bowtie2:2.5.4) were updated to include samtools for BAM conversion/sorting within the alignment tasks.

@tefirman tefirman marked this pull request as ready for review March 9, 2026 02:48
@tefirman tefirman requested a review from emjbishop as a code owner March 9, 2026 02:48
Copy link
Collaborator

@emjbishop emjbishop left a comment

Choose a reason for hiding this comment

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

Some minor suggestions

tefirman and others added 2 commits March 9, 2026 22:42
Co-authored-by: Emma Bishop <46635347+emjbishop@users.noreply.github.com>
@tefirman tefirman merged commit 431645a into main Mar 10, 2026
14 checks passed
@tefirman tefirman deleted the add-bowtie branch March 10, 2026 17:58
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