From 2685b443671d096691ee3a41be307e1b6f97ebbd Mon Sep 17 00:00:00 2001 From: daniel Date: Wed, 29 Nov 2023 15:43:15 +0100 Subject: [PATCH] fix: Remove default value definition in ConfigValidator. It is already done by CommonCLIParsingManager --- src/powerapi/cli/config_validator.py | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/src/powerapi/cli/config_validator.py b/src/powerapi/cli/config_validator.py index 65b62a07..674a1cb2 100644 --- a/src/powerapi/cli/config_validator.py +++ b/src/powerapi/cli/config_validator.py @@ -54,13 +54,6 @@ def validate(config: Dict): logging.error("no output configuration found") raise MissingArgumentException(argument_name='output') - for output_type in config['output']: - output_config = config['output'][output_type] - if 'model' not in output_config: - output_config['model'] = 'HWPCReport' - if 'name' not in output_config: - output_config['name'] = 'default_pusher' - if 'input' not in config: logging.error("no input configuration found") raise MissingArgumentException(argument_name='input') @@ -77,11 +70,6 @@ def validate(config: Dict): logging.error("stream mode cannot be used for csv input") raise NotAllowedArgumentValueException("Stream mode cannot be used for csv input") - if 'model' not in input_config: - input_config['model'] = 'PowerReport' - if 'name' not in input_config: - input_config['name'] = 'default_puller' - if 'pre-processor' in config: for pre_processor_id in config['pre-processor']: pre_processor_config = config['pre-processor'][pre_processor_id]