Skip to content

feat: --peOverlapNbasesMin mate-overlap merge-and-realign#119

Open
BenjaminDEMAILLE wants to merge 1 commit into
scverse:mainfrom
BenjaminDEMAILLE:bd/pe-overlap
Open

feat: --peOverlapNbasesMin mate-overlap merge-and-realign#119
BenjaminDEMAILLE wants to merge 1 commit into
scverse:mainfrom
BenjaminDEMAILLE:bd/pe-overlap

Conversation

@BenjaminDEMAILLE

Copy link
Copy Markdown
Contributor

Summary

  • Port of STAR's ReadAlign::peMergeMates / peOverlapSEtoPE (sister project STAR-rs star_pe_overlap.rs)
  • When mates overlap in genome space, merges them into one SE read, realigns through the existing SE pipeline, and converts the result back into a two-mate pair, rescoring from scratch
  • Unconditional overwrite of the separate-mate result on success, per STAR semantics

Test plan

  • cargo fmt --check
  • cargo clippy --all-targets (0 warnings)
  • cargo test (459 passed)

Port of STAR's ReadAlign::peMergeMates / peOverlapSEtoPE (STAR-rs
star_pe_overlap.rs): when mates overlap in genome space, merge them
into one SE read, realign through the existing SE pipeline, and
convert the result back into a two-mate pair, rescoring from scratch.
Unconditional overwrite of the separate-mate result on success, per
STAR semantics.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
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.

1 participant