Add methods for reading RNA sequences.#660
Conversation
|
There are some issues with the tests, most likely unrelated to this. I'll try restarting them again. |
|
I think the errors are with CATH tests (a NPE). I will run the tests locally to see if they pass. Also, could we add some tests for the new methods? Just copying the DNA tests and replacing them with RNA would be enough. |
|
Tests pass locally and I see that also in travis. Looks ok to me although would be nice to include the tests. |
|
Sure, I can add some tests. I was hunting for some Genbank RNA files in the wild but I couldn't find any, they aren't usually represented that way. |
|
Do you mean an RNA database? I know about RNAcentral for non-coding RNA sequences http://rnacentral.org Also NCBI seems to have mRNA and tRNA sequences, like https://www.ncbi.nlm.nih.gov/nuccore/AY892647.1 |
|
Specifically I meant Genbank sequences using the RNA alphabet. You can find FASTA formatted RNA sequences in the RNA alphabet in some of those databases, but Genbank RNA sequences (like AY892647.1 you linked) are represented in the DNA alphabet. |
|
True, I wonder why though! Seems that a simple U for T replacement is needed. |
|
So it seems that we can't have an RNA file for the test, right? I'll merge for the moment, but it'd be good to have a test at some point. |
Added methods for reading RNA sequences by copy paste search replace.