Option `groups` and `group` exist in verify.py

Indeed, there exist these two options. However, one of them, i.e., the --group option is a hidden parameter and is only used to internally.

Maybe you remember the structure of verify.py, i.e., that it calls itself when executing functions in the grid (or in parallel). There, it adds a certain set of new hidden (via argparse.SUPPRESS) command line options: https://gitlab.idiap.ch/bob/bob.bio.base/blob/master/bob/bio/base/script/verify.py#L22 that it uses to distinguish between the sub-tasks that it needs to perform (e.g., preprocessing, extraction, enrollment, scoring, ...). Some of these sub-tasks have additional options, and one of these options is --group.

Now, I can change this parameter to have a different name, but as it does not show up in the help message verify.py --help nor in the documentation: https://www.idiap.ch/software/bob/docs/latest/bioidiap/bob.bio.base/master/experiments.html#running-experiments-part-i I don't really see the need for that.

However, I can see that people by accident use the --group argument instead of --groups. As the latter has a default argument, it would silently swallow their argument without changing the behavior. Hence, a warning (in case the --group is specified w/o the presence of --sub-task) might be a good idea.

@mguenther: yes, we already had a case here of somebody that used --group instead of --groups and what you explained happened. The parser swallowed the option w/o outputting any error and the program did not behave as the user expected. Instead of a warning, shouldn't it be made an error? Anyways, any solution to avoid this common mistake from happening would be great.

mentioned in commit d92c8de1

mentioned in merge request !34 (merged)

mentioned in commit 33d85e00

mentioned in issue bob.bio.gmm#6 (closed)

Status changed to closed by commit 7a7b01d3

mentioned in commit 303ed8f2

Option `groups` and `group` exist in verify.py

Designs

Child items ...

Activity