Improve error message about iteration parameters size problem

populse · Oct 16, 2024 · 9ccc2c5 · 9ccc2c5
1 parent c9cea08
commit 9ccc2c5
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/capsul/pipeline/process_iteration.py b/capsul/pipeline/process_iteration.py
@@ -205,30 +205,30 @@ def _run_process(self):
                 if size is None:
                     size = psize
                 elif size != psize:
-                    size_error = True
+                    size_error = f'{parameter} (size {psize} != {size})'
                     break
                 if trait.output:
                     if no_output_value is None:
                         no_output_value = False
                     elif no_output_value:
-                        size_error = True
+                        size_error = f'{parameter} (size = {psize} but other parameter has no value)'
                         break
             else:
                 if trait.output:
                     if no_output_value is None:
                         no_output_value = True
                     elif not no_output_value:
-                        size_error = True
+                        size_error = f'{parameter} (size = {psize}, has no value)'
                         break
                 else:
                     if size is None:
                         size = psize
                     elif size != psize:
-                        size_error = True
+                        size_error = f'{parameter} (size {psize} != {size})'
                         break
 
         if size_error:
-            raise ValueError('Iterative parameter values must be lists of the same size: %s' % ','.join('%s=%d' % (n, len(getattr(self,n))) for n in iterative_parameters))
+            raise ValueError(f'Iterative parameter {size_error} values must be lists of the same size: %s' % ', '.join('%s=%d' % (n, len(getattr(self,n))) for n in iterative_parameters))
         if size == 0:
             return