macroevolution / bamm

A program for multimodel inference on speciation and trait evolution
GNU General Public License v2.0
33 stars 10 forks source link

Trees with polytomies give opaque warning "ERROR: tree std::string" #110

Closed SimonGreenhill closed 10 years ago

SimonGreenhill commented 10 years ago

I inadvertently gave bamm a tree with a polytomous branching. The error message was "ERROR: tree std::string" - which I think comes from Line 740 of Tree.cpp (and not, say, L748 or L756). After much puzzling I figured out that the tree was not binary. Passing the tree through R/ape's multi2di function allows bamm to work.

The attached pull request just makes this error message a bit more useful.

redcurry commented 10 years ago

Thanks!