summaryrefslogtreecommitdiff
path: root/utils/tuning
diff options
context:
space:
mode:
authorStefan Klug <stefan.klug@ideasonboard.com>2024-06-11 14:56:45 +0200
committerStefan Klug <stefan.klug@ideasonboard.com>2024-07-05 22:38:19 +0200
commit8e411bfaefaf931f849521149aa74151e1ccbd0c (patch)
tree84528400974e9277f432d2c6684324af403115b5 /utils/tuning
parent54a8405726e5d5aaddf0a44ee037fe544f31d836 (diff)
libtuning: Only warn if processing returns None
There are valid cases where a module returns None. E.g. no images were provided for LSC calibration. We should however define proper semantics there. Continue with a warning for now. Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Diffstat (limited to 'utils/tuning')
-rw-r--r--utils/tuning/libtuning/generators/yaml_output.py3
-rw-r--r--utils/tuning/libtuning/libtuning.py4
2 files changed, 5 insertions, 2 deletions
diff --git a/utils/tuning/libtuning/generators/yaml_output.py b/utils/tuning/libtuning/generators/yaml_output.py
index 31e265df..c490081d 100644
--- a/utils/tuning/libtuning/generators/yaml_output.py
+++ b/utils/tuning/libtuning/generators/yaml_output.py
@@ -107,6 +107,9 @@ class YamlOutput(Generator):
]
for module in output_order:
+ if module not in output_dict:
+ continue
+
out_lines.append(f' - {module.out_name}:')
if len(output_dict[module]) == 0:
diff --git a/utils/tuning/libtuning/libtuning.py b/utils/tuning/libtuning/libtuning.py
index 5342e5d6..e7c63535 100644
--- a/utils/tuning/libtuning/libtuning.py
+++ b/utils/tuning/libtuning/libtuning.py
@@ -200,8 +200,8 @@ class Tuner(object):
for module in self.modules:
out = module.process(self.config, images, self.output)
if out is None:
- logger.error(f'Module {module.hr_name} failed to process...')
- break
+ logger.warning(f'Module {module.hr_name} failed to process...')
+ continue
self.output[module] = out
self.generator.write(args.output, self.output, self.output_order)