Got it! I went through the code with the lowercasing stick. I left variable
name L as uppercase, though, for readability.


On Wed, Jan 23, 2013 at 10:07 AM, Anthony Scopatz
<notifications@github.com>wrote:

> In pyne/endf.py:
>
> > +        ZAI = line[0]
> > +        ABN = line[1]
> > +        LFW = line[3]
> > +        NER = line[4]
> > +        if line[2] == line[5] == 0:
> > +            if LFW in (1,0) and int(NER) == NER and ABN <= 1:
> > +                if ZAI >= 1000:
> > +                    return True
> > +        else:
> > +             return False
> > +
> > +    def is_range_line(self,line):
> > +        """ Determines whether this line is the beginning of a new resonance
> > +        range.  Line format must match EL, EH, LRU, LRF, NRO, NAPS. """
> > +
> > +        def check_NRO_NAPS(NRO, NAPS):
>
> PEP8 violations. function names should be lower case.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/pyne/pyne/pull/109/files#r2743604>.
>
>