Remove partial runnability checks in Pattern.add

thierry-martinez · thierry-martinez · commit 49616d8108e4 · 2025-11-20T11:15:42.000+01:00
diff --git a/graphix/pattern.py b/graphix/pattern.py
@@ -124,14 +124,11 @@ def add(self, cmd: Command) -> None:
             MBQC command.
         """
         if cmd.kind == CommandKind.N:
-            if cmd.node in self.__output_nodes:
-                raise RunnabilityError(cmd, cmd.node, RunnabilityErrorReason.AlreadyActive)
             self.__n_node += 1
             self.__output_nodes.append(cmd.node)
         elif cmd.kind == CommandKind.M:
-            if cmd.node not in self.__output_nodes:
-                raise RunnabilityError(cmd, cmd.node, RunnabilityErrorReason.AlreadyMeasured)
-            self.__output_nodes.remove(cmd.node)
+            if cmd.node in self.__output_nodes:
+                self.__output_nodes.remove(cmd.node)
         self.__seq.append(cmd)
 
     def extend(self, *cmds: Command | Iterable[Command]) -> None:
diff --git a/tests/test_pattern.py b/tests/test_pattern.py
@@ -692,13 +692,15 @@ def test_check_runnability_success(self, fx_rng: Generator) -> None:
         pattern.check_runnability()
 
     def test_check_runnability_failures(self) -> None:
+        pattern = Pattern(input_nodes=[0], cmds=[N(0)])
         with pytest.raises(RunnabilityError) as exc_info:
-            pattern = Pattern(input_nodes=[0], cmds=[N(0)])
+            pattern.check_runnability()
         assert exc_info.value.node == 0
         assert exc_info.value.reason == RunnabilityErrorReason.AlreadyActive
 
+        pattern = Pattern(cmds=[N(1), N(1)])
         with pytest.raises(RunnabilityError) as exc_info:
-            pattern = Pattern(cmds=[N(1), N(1)])
+            pattern.check_runnability()
         assert exc_info.value.node == 1
         assert exc_info.value.reason == RunnabilityErrorReason.AlreadyActive
 
@@ -720,8 +722,9 @@ def test_check_runnability_failures(self) -> None:
         assert exc_info.value.node == 0
         assert exc_info.value.reason == RunnabilityErrorReason.NotYetMeasured
 
+        pattern = Pattern(cmds=[N(1), M(1), M(1)])
         with pytest.raises(RunnabilityError) as exc_info:
-            pattern = Pattern(cmds=[N(1), M(1), M(1)])
+            pattern.check_runnability()
         assert exc_info.value.node == 1
         assert exc_info.value.reason == RunnabilityErrorReason.AlreadyMeasured
 
