In the LeresDetector it runs TestOptions().parse() which parses global sys.argv with abbreviations allowed.
When this package is used in environments such as Azure ML Endpoints there exists an --output sys arg that Leres then crashes on with the error error: ambiguous option: --output could match --output_nc, --output_dir, --output_resolution
Since this is a package that can be used in environments such as that, I suggest we turn off the allow abbreviations in the ArgumentParser
I think the default is fine when using from the command line on its own but as a package I think it should be configurable or set to False
This also exists in other classes but this is the one I'm concerned with.
What do you all think?
In the
LeresDetectorit runsTestOptions().parse()which parses globalsys.argvwith abbreviations allowed.When this package is used in environments such as Azure ML Endpoints there exists an
--outputsys arg that Leres then crashes on with the errorerror: ambiguous option: --output could match --output_nc, --output_dir, --output_resolutionSince this is a package that can be used in environments such as that, I suggest we turn off the allow abbreviations in the ArgumentParser
I think the default is fine when using from the command line on its own but as a package I think it should be configurable or set to
FalseThis also exists in other classes but this is the one I'm concerned with.
What do you all think?