Runtime parameters of mlcube can't include the letter "h" about mlcube HOT 3 CLOSED

Thanks @hasan7n for reporting this. Will look at it on Friday.

from mlcube.

@hasan7n Ok, so it should be fixed in #348.

Brief description of the cause of the behavior. We've gone through multiple rounds of refactoring of the MLCube command line interface, and at some point in time option descriptions moved to their module (cli) while deprecated option declarations remained in the __main__.py file. The secondary cause of this bug was that we updated APIs of most of MLCube CLI commands, but did not do that for the run command that continued to use the old interface. After updating its API to match APIs of other commands (such as configure), this bug went away.
The main cause however was the following. The run command did not expose any arguments for the MLCube parameters (e.g., -Pdocker.cpu_args="--shm-size 1g"). Turns out that the click library makes various attempts to parse options/arguments and match them with the long/short aliases of the arguments. The following was hapenning:

• The -Pdocker.cpu_args="--shm-size 1g" string is correctly split into name (-Pdocker.cpu_args) and value (--shm-size 1g).
• The click library was trying to match the name against known aliases (long names) of options, and was obviously failing.
• Then, it was trying to do fuzzy matching what also was resulting in empty matches.
• At this point in time (I have not spent too much time identifying the exact logic) the click library considers the entire string -Pdocker.cpu_args="--shm-size 1g as multiple short options, that seems to be consistent with other tools, e.g, one can run ls -al which is the same as ls -l -a. So, the click library removes the first character (-) and then iterates one character at a time considering options such as -P, -d, -o etc. and eventually -h. All options happen to be unknown except the latter, which is the cause for click to display help message.

As you correctly mention, when there's no h, this bug does not manifest itself.

from mlcube.

Associated PR #348 fixed this problem.

from mlcube.