-
Notifications
You must be signed in to change notification settings - Fork 426
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
Improved mapping results for a multi-part index - merging #189
Conversation
Thanks so much for this. I will review and merge later, definitely getting this ready for the next release 2.12. |
Learning from your code, I have reimplemented the support of multi-part index at the split branch. The new option is |
I have merged the split branch to master. I will close this PR. If you find bugs in the new feature, please create an issue. Thank you very much for this PR. It has been very helpful. |
The following features were added (for uni-segment reads only). The additions were done such that minimal changes are done on existing code.
Tested using two datasets 33k reads and 3.9M synthetic long reads. Tested to confirm that the changes do not affect the existing result (see the attached evaluate_accuracy.zip)
evaluate_accuracy.zip