Bug in U/WPGMA!

4 views
Skip to first unread message

Julien Yann Dutheil

unread,
Sep 7, 2012, 3:30:42 AM9/7/12
to biopp-de...@googlegroups.com
Hi there,

Just to mention i have found a bug in the PGMA class. To fix it, I had to update a bit the interface of the DistanceMethod class. I updated bppDist accordingly, but these changes may lead to broken code if you were using the class, sorry about that. The update is not complicated, the only difference is that the "rooted" argument is now passed to the constructor of the class and not to the computeTree method anymore, as some method like U/WPGMA output a rooted tree. Using computeTree(false) on U/WPGMA therefore led to an incorrect result.

All the best and sorry again for the inconvenience, if any.

Julien.
Reply all
Reply to author
Forward
0 new messages