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

that poor sucker
Good eye!

On Tue, Sep 16, 2014 at 7:13 PM, Mario Giovacchini <
mario.giovacchini@scilifelab.se> wrote:

> 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>
>> .
>>
>
>