Yes, but named to be consistent with "add_spec_node", which adds an implied
speciation node.


On Sun, Apr 6, 2014 at 8:14 PM, Matt Rasmussen <notifications@github.com>wrote:

> In compbio/phylo.py:
>
> > @@ -1139,6 +1264,37 @@ def add_spec_node(node, snode, tree, recon, events):
> >
> >      return newnode
> >
> > +def remove_spec_node(node, tree, recon=None, events=None):
> > +    """
> > +    removes speciation node 'node' from gene tree 'tree'
>
> Do you mean remove an *implied* speciation node? The assert for len(node.children)
> == 1 makes me think this is what you intend.
>
> Given this interpretation, function looks good.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/wutron/compbio/pull/2/files#r11328901>
> .
>