Skip to content

[codex] Merge overlapping fragment mates during assembly#176

Merged
iskandr merged 1 commit intomasterfrom
codex/merge-overlapping-fragments
Apr 23, 2026
Merged

[codex] Merge overlapping fragment mates during assembly#176
iskandr merged 1 commit intomasterfrom
codex/merge-overlapping-fragments

Conversation

@iskandr
Copy link
Copy Markdown
Contributor

@iskandr iskandr commented Apr 23, 2026

Summary

  • conservatively merge overlapping paired-end reads from the same fragment when run_isovar builds read evidence for assembly
  • preserve raw read counts with source_read_count so count fields and adapter evidence stay numerically correct
  • keep raw read and CLI surfaces unchanged by making fragment merging opt-in on ReadCollector
  • add regressions for the merged fragment sequence and preserved raw read counts
  • bump isovar to 1.5.1

Why

Issue #59 asks Isovar to stop treating overlapping mate pairs as independent evidence when assembling cDNA around a variant. That duplicated overlap can fragment assembly and overstate support in downstream assembled-sequence metrics.

Validation

  • ./lint.sh
  • ./test.sh

Closes #59

@iskandr iskandr marked this pull request as ready for review April 23, 2026 18:27
@iskandr iskandr merged commit 81d142b into master Apr 23, 2026
6 checks passed
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 93.113% (-0.2%) from 93.305% — codex/merge-overlapping-fragments into master

@iskandr iskandr deleted the codex/merge-overlapping-fragments branch April 23, 2026 18:31
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.

Merge overlapping read pairs into single LocusRead

2 participants