Skip to content

Bug/Suggestion: Don't Parse Global Sys Args With Allowing Abbreviations #121

@ewwwgiddings

Description

@ewwwgiddings

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?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions