So in future, use underscores?  I think fix_ils_errors is used in only a
few places, so I can make the necessary changes w/o breaking API.


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

> In compbio/phylo.py:
>
> > @@ -540,27 +625,62 @@ def walk(node):
> >      return conf
> >
> >
> > +def fix_ils_errors(events, dupcons, newCopy=True):
>
> New variables should use underscores like new_copy. Unfortunately some of
> my old code uses names like newCopy which I have been hesitant to change
> for fear of breaking API.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/wutron/compbio/pull/2/files#r11328803>
> .
>