Skip to content
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

fix: Fix read processing when building consensus reads #199

Merged
merged 6 commits into from
Dec 1, 2021

Conversation

FelixMoelder
Copy link
Member

@FelixMoelder FelixMoelder commented Nov 24, 2021

In some cases consensus read calculation fails as read groups may be processed multiple times.
To fix this two major changes have been introduced:

  • Read containg softclips will now be skipped and written to the output bam-file directly.
  • A new struct GroupEndIndex has been introduced for processing complete read groups.
    The struct manages the end positions of read groups.

Feature:
Existing UMIs will be read from the RX-Tag and added to the description of the consensus reads. These can be converted into actual bam tags by bwa mem (Adding option -C during mapping).

@johanneskoester johanneskoester merged commit 3fdcb3f into master Dec 1, 2021
@johanneskoester johanneskoester deleted the consensus_fix_group_processing branch December 1, 2021 12:02
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.

None yet

2 participants