fix: update dependencies and add missing tools (closes #11)#21
Closed
Jaureguy760 wants to merge 7 commits intomasterfrom
Closed
fix: update dependencies and add missing tools (closes #11)#21Jaureguy760 wants to merge 7 commits intomasterfrom
Jaureguy760 wants to merge 7 commits intomasterfrom
Conversation
Major changes from 194 commits of development work: ## New Features - Rust acceleration for BAM counting (7x speedup) - Unified FASTQ/BAM pipeline - Single-cell support (scRNA-seq, scATAC-seq) - Enhanced statistical analysis (beta-binomial) - Improved CLI with better argument handling ## New Files - rust/ - Rust source for BAM counting acceleration - tests/ - Comprehensive test suite - docs/ - Enhanced documentation - .github/workflows/ - CI configuration - pyproject.toml - Modern Python packaging - pytest.ini, mypy.ini - Quality tools ## Module Updates - src/counting/ - Rust-accelerated allele counting - src/mapping/ - Optimized read mapping - src/analysis/ - Enhanced statistical methods - src/wasp2/ - Unified CLI entry points Closes #20 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Sync version to 1.2.0 across pyproject.toml, Cargo.toml, __init__.py - Fix Rust panic on empty allele strings in bam_counter.rs (use unwrap_or) - Fix Rust panic on file create in unified_pipeline.rs (use ? with context) - Fix Rust panic on HashMap get in bam_remapper.rs (use ok_or_else) - Fix bare except block in run_mapping.py (catch KeyError specifically) - Add error tracking for silent BAM failures in mapping_filter.rs - Add failure tracking and warnings in make_remap_reads.py Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add BAM read error tracking in bam_counter.rs (log first 5, warn total) - Add logging and error tracking in count_alleles.py - Convert test_rust_python_match.py to proper pytest with fixtures - Convert test_indel_correctness.py to proper pytest with fixtures Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Updates dependency configuration to address Issues #3, #4, and #6: - Add missing bioinformatics tools: samtools, bcftools, htslib, bedtools - Fix anndata/pandas version compatibility (anndata>=0.10.0,<0.12.0) - Add scientific libraries: statsmodels, scanpy, pyarrow, h5py - Synchronize versions across environment.yml, requirements.txt, pyproject.toml - Add Python 3.12 support in classifiers and black target-version - Create requirements-dev.txt for development dependencies - Update README.md with correct Python 3.10+ requirement Implemented via velocity workflow. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Adds <2.0 upper bound to maturin in [project.optional-dependencies].dev to match the constraint in rust optional deps and requirements-dev.txt. This prevents potential build breakage if maturin 2.x is released with breaking changes. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Collaborator
Author
|
Moving to Jaureguy760/WASP2-exp |
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
1 participant
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.
Summary
anndata>=0.10.0,<0.12.0Test plan
conda env create -f environment.ymlsucceedspip install -r requirements.txtsucceedspip install -r requirements-dev.txtsucceedspytest tests/to confirm tests passCloses #11, #3, #4, #6
🤖 Generated with Claude Code