Project

General

Profile

Bug #2393

incorrect error message with omitted command line flag before its argument

Added by Szilárd Páll almost 2 years ago. Updated almost 2 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
mdrun
Target version:
-
Affected version - extra info:
Affected version:
Difficulty:
simple
Close

Description

Ran the following command:

gmx mdrun -v -resethway -noconfout -pin on -ntmpi 1 -ntomp 16 -nsteps 10000 -nb gpu -pme cpu -notunepme test_1x16_2TPC_nb-cpu_pme-cpu

(note the missing -g).

Got the following message:

Error in user input:
Invalid command-line options
  In command-line option -notunepme
    Invalid value: 'test_1x16_2TPC_nb-cpu_pme-cpu'; supported values are: 1,
    0, yes, no, true, false
  In command-line option -notunepme
    Cannot specify a value together with 'no' prefix

For more information and tips for troubleshooting, please check the GROMACS
website at http://www.gromacs.org/Documentation/Errors

The first message seems to be incorrect and should never even appear, I think.

History

#1 Updated by Berk Hess almost 2 years ago

I tried to debug this, but then tried values 1, 0, yes, no, true, false for -tunepme and they actually all work. This seems to be the case since version 5.1. I didn't now this.
So although two error messages is somewhat confusing, it looks like it's correct.

#2 Updated by Szilárd Páll almost 2 years ago

I knew that -notunepme is synonymous with -tunepme no, but the first message is still incorrect as it claims that e.g. -notunepme yes is correct which is false.

#3 Updated by Mark Abraham almost 2 years ago

Perhaps the best error message would be

  In command-line option -tunepme
    Cannot specify a value 'test_1x16_2TPC_nb-cpu_pme-cpu' 
    together with 'no' prefix. Either use the prefix and no value, or no prefix
    and supply a supported value from: 1, 0, yes, no, true, false

but actually fixing the old parse_common_args() gear to do this is a much lower priority than porting those options to newer infrastructure.

Also available in: Atom PDF