No preference
On May 27, 2014 10:16 AM, "Greg Caporaso" <notifications@github.com> wrote:

> In skbio/parse/sequences/fastq.py:
>
> > @@ -158,10 +157,11 @@ def parse_fastq(data, strict=False, phred_offset=33):
> >              # http://nar.oxfordjournals.org/content/38/6/1767/T1.expansion.html
> >              elif linetype == QUAL:
> >                  qual = phred_f(line)
> > -                if (qual < 0).any() or (qual > 62).any():
> > -                    raise FastqParseError("Failed qual conversion for seq "
> > -                                          "id: %s. This may be because you "
> > -                                          "passed an incorrect value for "
> > -                                          "phred_offset." % seqid)
> > +                if enforce_qual_range:
> > +                    if (qual < 0).any() or (qual > 62).any():
>
> Why not and instead of nested if statement? Just a readability thing - I
> prefer and but don't worry about it if you guys disagree.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/biocore/scikit-bio/pull/394/files#r13086342>
> .
>