I do not envy whoever has to clean all this up someday

that poor sucker
Yeah that was a leftover actually, I didn't catch it then but I have since
removed it. I'll push changes

On Tue, Sep 16, 2014 at 4:10 PM, Guillermo Carrasco <
notifications@github.com> wrote:

> In scripts/start_pipeline_from_project.py:
>
> > +    project = recreate_project_from_filesystem(demux_fcid_dir,
> > +                                               restrict_to_projects,
> > +                                               restrict_to_samples)
> > +    launch_analysis_for_flowcells([project])
> > +
> > +
> > +if __name__ == '__main__':
> > +    parser = argparse.ArgumentParser()
> > +    parser.add_argument("-p", "--project", dest="restrict_to_projects", action="append",
> > +            help=("Restrict processing to these projects. "
> > +                  "Use flag multiple times for multiple projects."))
> > +    parser.add_argument("-s", "--sample", dest= "restrict_to_samples", action="append",
> > +            help=("Restrict processing to these samples. "
> > +                  "Use flag multiple times for multiple samples."))
> > +    parser.add_argument("demux_fcid_dir", nargs="?", action="store",
> > +            default="/proj/a2010002/nobackup/mario/DATA/140528_D00415_0049_BC423WACXX/",
>
> do you think it's a good idea to add a default to a required arg? Even
> more, a default that will be unaccessible for anyone but yourself?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NationalGenomicsInfrastructure/ngi_pipeline/pull/89/files#r17602805>
> .
>