-
Notifications
You must be signed in to change notification settings - Fork 18
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
Joining nearby NAMs for combined score #415
Comments
In the above comment, I had benchmarked with asymmetric seeds and concluded no significant advantage. Asymmetric seeds currently have other issues, which doesn't give a clean benchmark. Therefore, I decided to implement the above described 'slack' based off the main branch (which uses symmetric seeds). Below I provide a benchmark on a small chrX+Y dataset using
In single end mode I also include I observe quite drastic accuracy improvements on all but the 50nt reads with the It should be carefully noted that this is a small dataset. The rescue might do a better job on a full human genome(?). PAIRED END READS
SINGLE END READS
(CC @Itolstoganov) |
The idea of joining nearby NAMs for increased score/power and to reduce redundant alignments seem logical. However, it hasn’t worked as effectively as expected. I found one reason, that I describe in this comment: 3d96321#commitcomment-140411641
Even after fixing as above, the idea is still not giving a great benefit on a smaller benchmark, but there could be remaining issues with the logic, or the testing I did is not representative of what happens on the main branch implementation (described in comment).
The text was updated successfully, but these errors were encountered: