sumo icon indicating copy to clipboard operation
sumo copied to clipboard

unknown options are no longer reported

Open behrisch opened this issue 3 years ago • 3 comments

~/sumo/tools/randomTrips.py --blubber blub -n ~/sumo/tests/sumo/net.net.xml just runs but it should complain

behrisch avatar Jul 27 '22 13:07 behrisch

The breaking commit was d7e05e1d722333b670c1509e3c9b7ec2e34a3784 (1.14.1)

behrisch avatar Jul 28 '22 09:07 behrisch

This is not completed yet. The current option parsing is still too generous because it still accepts an option like --foo-bar baz but at least not --foo baz any longer. I would propose that the call to parse_args (or the constructor of the ArgumentParser) gets another parameter which defines the program prefixes to accept. Alternatively (and less preferred) we could hardcode the common SUMO programs (like sumo, duarouter etc.) Please discuss!

behrisch avatar Jul 28 '22 11:07 behrisch

I'd prefer an extra option for ArgumentParser.

namdre avatar Aug 08 '22 13:08 namdre

@namdre I would like to solve this issue.

bcoueraud87 avatar Aug 15 '22 11:08 bcoueraud87

fixed in #11368

namdre avatar Aug 16 '22 13:08 namdre

also creates problems with duaIterate.py (but a different error)

namdre avatar Aug 25 '22 13:08 namdre