search

mila-iqia / platoon

Multi-GPU mini-framework for Theano
MIT License
195 stars 41 forks source link

Watch all processes for premature exit. #51

Closed abergeron closed 8 years ago

abergeron commented 8 years ago

Should fix #42

abergeron commented 8 years ago

Is there anything else for this?

abergeron commented 8 years ago

Oh, yeah, I just remembered about the -c failure, I'll get to that.

abergeron commented 8 years ago

Ok so argparse insists that anything that starts with a dash (with the exception of strings that contain spaces) must absolutely be an option and cannot ever be an argument to something under any possible circumstance.

Therefore we must work around its stupidity and instruct users to do so.

mgermain commented 8 years ago

How about rolling back to the way it was? I think it was working with argpars.

On Fri, Feb 5, 2016 at 1:39 PM, abergeron notifications@github.com wrote:

Ok so argparse insists that anything that starts with a dash (with the exception of strings that contain spaces) must absolutely be an option and cannot ever be an argument to something under any possible circumstance.

Therefore we must work around its stupidity and instruct users to do so.

— Reply to this email directly or view it on GitHub https://github.com/mila-udem/platoon/pull/51#issuecomment-180492553.

abergeron commented 8 years ago

No it wasn't. You only got lucky because your example has a space in the middle.

Try -c --do-syncwith the current code (and you can add quoting it still won't work).

abergeron commented 8 years ago

Rebased on top of the python 3 changes.