It's fine for this application, yes.

On Tue, May 20, 2014 at 12:54:35PM -0700, Michael R. Crusoe wrote:
> >  
> > -    sequence = record['sequence']
> > -    name = record['name']
> > +def get_parser():
> > +    parser = argparse.ArgumentParser(
> > +        description='Convert fastq (.fq) files to fasta (.fa).',
> > +        formatter_class=argparse.ArgumentDefaultsHelpFormatter)
> > +
> > +    parser.add_argument('input', help='The name of the input'
> > +                        ' FASTQ sequence file.')
> > +    parser.add_argument('-o', '--output', help='The name of the output'
> > +                        ' FASTA sequence file.')
> > +    parser.add_argument('-n', '--n_keep', default=False, action='store_true',
> 
> I see that we're defaulting to keeping reads with Ns in them. @CTB does that make sense?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ged-lab/khmer/pull/426/files#r12864428
-- 
C. Titus Brown, ctb@msu.edu