Repository navigation
bwamem3/index, bwamem3/mem: bump bwa-mem3 to 0.17.0 - #13715
Merged
Merged
Conversation
0.17.0 is faster and uses less memory under --meth. Bare --meth (bisulfite/EM-seq) now defaults to --meth-scoring genomic; the mem meta.yml notes how to keep the 0.16.0 scoring. --meth=taps and plain DNA output, and the index, are unchanged. Generated by Claude Code Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
11 tasks done
eduard-watchmaker
marked this pull request as ready for review
November 12, 2026 20:39
eduard-watchmaker
enabled auto-merge
November 12, 2026 20:42
heuermh
approved these changes
Nov 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps
bwamem3/indexandbwamem3/memto bwa-mem3 0.17.0, as suggested by @nh13 on nf-core/methylseq#638. It is faster and uses less memory under--meth.Output changes:
--meth(bisulfite/EM-seq): scoring now defaults to--meth-scoring genomic(feat(meth)!: default EM-seq/bisulfite --meth-scoring to genomic fg-labs/bwa-mem3#756), so scores, MAPQ, NM and some placements change. The module tests record read sequences and headers rather than scores, so their snapshots only change in the version strings.ext.args = '--meth --meth-scoring collapsed'keeps the 0.16.0 output; this is now noted in thememmeta.yml.--meth=tapsand plain DNA alignment: unchanged.--meth, so existing indexes keep working.-u: alternative hits are now written asXB:Zinstead ofXA:Z. No nf-core component passes-u.Changes
bwamem3/index,bwamem3/mem: bwa-mem3 0.16.0 → 0.17.0, with new Seqera containers, conda lock files and meta.yml container entries (nf-core modules containers create).bwamem3/memmeta.yml: note on the new bare--methscoring default and how to keep the old one.fasta_index_dna,fasta_index_methylseq,fastq_align_dedup_bwamem3andfastq_align_dna) pass against their existing snapshots, unchanged.Generated by Claude Code
PR checklist
versions.ymlfile.labelnf-core modules test bwamem3/index --profile dockernf-core modules test bwamem3/mem --profile dockernf-core subworkflows test fasta_index_dna --profile dockernf-core subworkflows test fasta_index_methylseq --profile dockernf-core subworkflows test fastq_align_dedup_bwamem3 --profile dockernf-core subworkflows test fastq_align_dna --profile docker🤖 Generated with Claude Code