Skip to content

Commit

Permalink
Merge pull request #216 from powerapi-ng/fix/issue215/management-mode…
Browse files Browse the repository at this point in the history
…l-parameter-puller-pusher

fix: Remove default value definition in ConfigValidator. It is alread…
  • Loading branch information
gfieni authored Nov 30, 2023
2 parents 15eab7c + 2685b44 commit d6532f1
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions src/powerapi/cli/config_validator.py
Original file line number Diff line number Diff line change
Expand Up @@ -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')
Expand All @@ -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]
Expand Down

0 comments on commit d6532f1

Please sign in to comment.