From a429f83f5e7b54047f15f5eb3fef9766dcc00c3e Mon Sep 17 00:00:00 2001 From: hroff-1902 Date: Tue, 23 Apr 2019 22:16:24 +0300 Subject: [PATCH] flake happy; check_positive() renamed --- freqtrade/arguments.py | 10 ++++++---- freqtrade/configuration.py | 3 ++- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/freqtrade/arguments.py b/freqtrade/arguments.py index ce84d7425..8ae3922ca 100644 --- a/freqtrade/arguments.py +++ b/freqtrade/arguments.py @@ -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: diff --git a/freqtrade/configuration.py b/freqtrade/configuration.py index 9994d405d..ca3b81279 100644 --- a/freqtrade/configuration.py +++ b/freqtrade/configuration.py @@ -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