Skip to content

feat: Add bam::Record::set_cigar #477

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

ctsa
Copy link
Contributor

@ctsa ctsa commented Jun 21, 2025

This is largely modeled off of the existing bam::Record::set_qname method (including its unit test).

The motivation is similar: it's convenient to be able to update just the cigar string of an existing alignment in some cases, without having to re-run much of the bam record data through the set() method.

@ctsa ctsa changed the title Add bam::Record::set_cigar feat: Add bam::Record::set_cigar Jun 21, 2025
@coveralls
Copy link

coveralls commented Jun 21, 2025

Pull Request Test Coverage Report for Build 15793975163

Details

  • 23 of 23 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 83.724%

Totals Coverage Status
Change from base Build 15455217694: 0.1%
Covered Lines: 2747
Relevant Lines: 3281

💛 - Coveralls

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