flake happy; check_positive() renamed

This commit is contained in:
hroff-1902 2019-04-23 22:16:24 +03:00
parent fc4ef2b430
commit a429f83f5e
2 changed files with 8 additions and 5 deletions

View File

@ -311,7 +311,7 @@ class Arguments(object):
help='Set random state to some positive integer for reproducible hyperopt results.',
dest='hyperopt_random_state',
default=None,
type=Arguments.check_positive,
type=Arguments.check_int_positive,
metavar='INT',
)
@ -385,13 +385,15 @@ class Arguments(object):
raise Exception('Incorrect syntax for timerange "%s"' % text)
@staticmethod
def check_positive(value) -> int:
def check_int_positive(value) -> int:
try:
uint = int(value)
if uint <= 0:
raise ValueError
except:
raise argparse.ArgumentTypeError(f"{value} is invalid for this parameter, should be a positive integer value")
except ValueError:
raise argparse.ArgumentTypeError(
f"{value} is invalid for this parameter, should be a positive integer value"
)
return uint
def scripts_options(self) -> None:

View File

@ -315,7 +315,8 @@ class Configuration(object):
if 'hyperopt_random_state' in self.args and self.args.hyperopt_random_state is not None:
config.update({'hyperopt_random_state': self.args.hyperopt_random_state})
logger.info("Parameter --random-state detected: %s", config.get('hyperopt_random_state'))
logger.info("Parameter --random-state detected: %s",
config.get('hyperopt_random_state'))
return config