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

Multithread bam read #812

Merged
merged 3 commits into from
Nov 13, 2024
Merged

Conversation

rhpvorderman
Copy link
Collaborator

Pending. This works on marcelm/dnaio#141. We might want to add additional functionality to dnaio 1.3.0 for other uBAM support features.

@rhpvorderman rhpvorderman marked this pull request as ready for review November 12, 2024 11:03
@rhpvorderman
Copy link
Collaborator Author

We might want to add additional functionality to dnaio 1.3.0 for other uBAM support features.

This is still on my to-do list, but not very high. The adapter ends in ONT Nanopore don't interfere with alignment at all and seem to have very little effect (within 1% error-margin) on variant calling. So the priority to allow proper methylation tag support when cutting has somewhat lessened.

Currently I have a lot of work to do so I will pick that up at a later date. For now I have made sure that multithreaded BAM reading works with the latest version of dnaio so that it can be included into the new version of cutadapt. Hence the @marcelm ping, so you know it is ready :-).

@marcelm
Copy link
Owner

marcelm commented Nov 13, 2024

Looks good to me!

@marcelm marcelm merged commit 628ce98 into marcelm:main Nov 13, 2024
16 checks passed
@rhpvorderman rhpvorderman deleted the multithread_bam_read branch November 13, 2024 11:03
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