Cassiopeia icon indicating copy to clipboard operation
Cassiopeia copied to clipboard

Generalizing 'collapse_mutationless_edges' and small fitchcount bugfix

Open richardyz98 opened this issue 2 years ago • 1 comments

This pull request includes the generalization of the collapse_mutationless_edges function. This allows us to use a function that does not factor in missing data when determining whether to collapse two nodes. If we do not collapse on missing data, then the baseline score increases for trees as missing data increases.

Additionally, it fixes a small bug in fitchcount which can error if no root is provided, such as on an initial call of the function.

richardyz98 avatar May 25 '22 16:05 richardyz98