Improved Arg Parsing

[AliSot2000]

Hi again

FYI, you can improve the argparsing you're doing by using the action parameter of add_argument.

Example:

parser.add_argument('-la', '--lazy', type=lambda x: bool(_help._strtobool(x)), help='Compares image dimensions before comparison process.', required=False, choices=[True, False], default=True)    

If you provide -la or --lazy you must provide a True or False otherwise the parser will complain that there's no argument for the flag.

dif.py: error: argument -la/--lazy: expected one argument 

Try this instead:

parser.add_argument('-la', '--lazy', help='Compares image dimensions before comparison process.', action='store_true')

The action store_true will store True in the namespace when the flag is provided, it sets required to False, sets the type to bool and sets the default to False

If you want a flag which disables something like rotate you can use the inverse store_false which just inverts the default and const so:

parser.add_argument('-ro', '--rotate', action="store_False")

[elisemercury]

Hi @AliSot2000,

Thanks a lot for the suggestion! We will potentially consider your suggestion this as part of future upgrades.

Best,
Elise