I'm good with reverting the code (but keeping the note in the doc-string).
 I was comparing true reconciliations (in simulation) to inferred (MPR)
reconciliations and noticed some cases in which events were not being
inferred due to nodes being mapped further up in the tree.  So I put the
commented code to try and generalize and handle non-MPR cases, but it is
not fully tested.


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

> In compbio/phylo.py:
>
> >      loss = []
> >
> >      def walk(node):
> >          loss.extend(find_loss_node(node, recon))
> > -        node.recurse(walk)
> > +
> > +        # add losses (for non-MPR)
>
> If this code is not ready yet, can we revert it?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/wutron/compbio/pull/2/files#r11328766>
> .
>