Merge pull request #39 from Aljaz-R/styling

Added another check to see if pipeline is set up correctly.

Author: firefly-cpp

niaaml_gui/main.py

@@ -66,6 +66,27 @@ def __init__(self):
         self.errorMessage.setStandardButtons(QMessageBox.StandardButton.Ok)
 
     def run_pipeline(self):
+        
+        invalid_blocks = []
+
+        for block, info in self.pipelineCanvas.block_data.items():
+            label = info.get("label", "")
+            if hasattr(block, "get_value"):
+                value = block.get_value()
+            elif hasattr(block, "dropdown"):
+                value = block.dropdown.currentText()
+            else:
+                value = info.get("path") or getattr(block, "value", None)
+
+            if not value or (isinstance(value, str) and not value.strip()):
+                invalid_blocks.append(label)
+
+        if invalid_blocks:
+            msg = "Pipeline isn't set up correctly. Missing components:\n\n" + "\n".join(f"- {label}" for label in invalid_blocks)
+            self.errorMessage.setText(msg)
+            self.errorMessage.exec()
+            return
+        
         blocks = self.pipelineCanvas.block_data
         if not blocks:
             self.errorMessage.setText("Pipeline is empty!")
@@ -175,7 +196,7 @@ def validate_pipeline_ready(self):
                 value = block.dropdown.currentText()
             else:
                 value = info.get("path") or getattr(block, "value", None)
-
+                            
             if not value or (isinstance(value, str) and not value.strip()):
                 all_valid = False
                 break

tests/test_pipeline_canvas.py

@@ -20,8 +20,6 @@ def test_pipeline_ready_false_when_empty(canvas):
     canvas.block_data.clear()
     canvas.scene.clear()
 
-    print("Block data keys:", canvas.block_data.keys())
-
     assert len(canvas.block_data) == 0
     assert not canvas.is_pipeline_ready()