Skip to content

v0.2.0#1

Merged
dialvarezs merged 16 commits intomainfrom
dev
Nov 23, 2025
Merged

v0.2.0#1
dialvarezs merged 16 commits intomainfrom
dev

Conversation

@dialvarezs
Copy link
Copy Markdown
Owner

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces v0.2.0 with significant enhancements to readfaker, including BAM format support, multi-base indel error modeling, and a dependency migration from needletail/bgzip to noodles for improved file I/O handling.

  • Adds BAM file support for both input (model extraction) and output (read generation)
  • Implements geometric distribution for multi-base insertions and deletions with configurable extension rates
  • Migrates from needletail/bgzip to noodles library for unified FASTA/FASTQ/BAM handling

Reviewed changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/utils.rs Added BAM support to load_models with automatic format detection based on file extension
src/models/error.rs Added insertion/deletion extension rates with geometric distribution sampling for multi-base indels
src/main.rs Integrated error model CLI parameters and added output format detection (BAM vs FASTQ)
src/io/mod.rs Added BAM module and re-exported BAM reader/writer types
src/io/fastq.rs Migrated from needletail to noodles, improved compression handling with configurable thread count
src/io/fasta.rs Migrated from needletail to noodles for FASTA reading
src/io/bam.rs New module implementing BAM file reading and writing with proper quality score encoding
src/generator.rs Updated to work with noodles FASTQ record types
src/cli.rs Added CLI parameters for error model configuration and compression threads
Cargo.toml Replaced needletail/bgzip with noodles and flate2 dependencies
Cargo.lock Updated dependency tree reflecting noodles migration
.gitignore Added AI agent directories to gitignore

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/cli.rs Outdated
Comment thread src/utils.rs Outdated
Comment thread src/main.rs Outdated
Comment thread src/io/mod.rs Outdated
Comment thread src/io/fastq.rs
Comment thread src/models/error.rs
@dialvarezs dialvarezs merged commit a5d8259 into main Nov 23, 2025
1 check passed
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