Skip to content

perf(compare): replace RecordBuf with raw byte comparison for compare-bams - #197

Merged
nh13 merged 1 commit into
mainfrom
nh/faster-compare-bams
Mar 28, 2026
Merged

nh13 merged 1 commit into
mainfrom
nh/faster-compare-bams

perf(compare): replace RecordBuf with raw byte comparison for compare…

a3db899
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Mar 28, 2026 in 1s

87.84% (+1.71%) compared to 7442efd

View this Pull Request on Codecov

87.84% (+1.71%) compared to 7442efd

Details

Codecov Report

❌ Patch coverage is 86.09524% with 73 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.84%. Comparing base (7442efd) to head (a3db899).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/commands/compare/bams.rs 72.15% 71 Missing ⚠️
src/commands/compare/raw_compare.rs 99.25% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #197      +/-   ##
==========================================
+ Coverage   86.12%   87.84%   +1.71%     
==========================================
  Files         110      111       +1     
  Lines       51974    52217     +243     
==========================================
+ Hits        44765    45870    +1105     
+ Misses       7209     6347     -862     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.