RNA alphabet

7 views
Skip to first unread message

Julien Yann Dutheil

unread,
Oct 3, 2012, 2:59:20 PM10/3/12
to biopp-de...@googlegroups.com
Hi all (but most particularly Sylvain :D)

I have found that the method DNA::getAlias(int) did not work anymore with gaps, that is, getAlias(-1) returned an empty vector, while it returns -1 in proteins for instance (and that is the behavior we want... this was the cause of the bug in the parsimony class, see the help forum).
I made a small modification to account for that, but maybe there is a better way to do so using the binary codes that Sylvain introduced, so maybe you may want to double-check the code I modified...
Also, I realized that all the modifications carried on by Sylvain using the "binary codes" were only present in DNA and not in RNA. I cannot see any reason for that, so I duplicated all the code in DNA to the RNA class. Sylvain, is there a problem in doing so? Could that code go in a common class instead of being duplicated in both DNA and RNA, or am I missing/forgetting something?

Cheers,

Julien.

Sylvain Gaillard

unread,
Oct 5, 2012, 3:54:05 AM10/5/12
to biopp-de...@googlegroups.com
Hi Julien,
I'll take a look at that asap ... but time is a precious thing that I
miss currently ;-)
Sylvain
> --
> You received this message because you are subscribed to the Google
> Groups "Bio++ Development Forum" group.
> To post to this group, send email to biopp-de...@googlegroups.com.
> To unsubscribe from this group, send email to
> biopp-devel-fo...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msg/biopp-devel-forum/-/TgEr5h033lAJ.
> For more options, visit https://groups.google.com/groups/opt_out.
>
>

Sylvain Gaillard

unread,
Oct 5, 2012, 8:17:41 AM10/5/12
to biopp-de...@googlegroups.com
I've taken the time to look at your modifications. That's OK for me.
Duplication of code in DNA and RNA is not a problem as these classes are
independent, they just inherit form the NucleicAlphabet class and
provide two distinct classes used in different context. I don't think
that the code had to be shared just for ease of coding as they are two
distinct classes that happen to have the same implementation because of
their similarities in term of biochemistry.
Best,
Sylvain

Julien Yann Dutheil

unread,
Oct 5, 2012, 10:16:57 AM10/5/12
to biopp-de...@googlegroups.com
Ok Sylvain, thanks a lot.

We leave it as is (at least for now).

Cheers,

J.
--
Julien Y. Dutheil, Ph-D
0 (+49) 6421 178 530

§ Max Planck Institute for Terrestrial Microbiology
Department of Organismic Interactions
Marburg -- GERMANY

§ Intitute of Evolutionary Sciences - Montpellier
University of Montpellier 2 -- FRANCE
Reply all
Reply to author
Forward
0 new messages