Support for RNA #33
Labels
Comments
lh3
added a commit
that referenced
this issue
Oct 7, 2017
|
Implemented via bf2d4f7. For the minimap2 command line, one change to sequence reading is sufficient. However, for API users who don't use bseq.c, we also need to change the char to base table |
|
Ah, right. Thanks for the quick implementation! |
|
I am closing this for now. Let me if you have issues with this change. Thanks for the info. Didn't know RNA reads use "U". |
|
No problem, thanks. I didn't have time to test it today but I'll reopen if
I run into any problems.
Jared
…On Tue, Oct 10, 2017 at 9:39 PM, Heng Li ***@***.***> wrote:
Closed #33 <#33>.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#33 (comment)>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AAXxn_7Gl4UVBUpkVVHj22ODfW3M3CuIks5srBxQgaJpZM4PxXV8>
.
|
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @lh3,
I'd like to add support for direct RNA nanopore reads to minimap2. These reads have Us in their basecalled sequence rather than Ts. I'm happy to provide a PR but wanted to run the proposed solution by you first - is it enough to convert Us to Ts after reading the sequence here or would you prefer I do something else?
Jared
The text was updated successfully, but these errors were encountered: