refactoring: fixed natural_transition set property

aiplan4eu · Dec 3, 2024 · c770833 · c770833
1 parent 1ec433c
commit c770833
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/unified_planning/model/mixins/natural_transitions_set.py b/unified_planning/model/mixins/natural_transitions_set.py
@@ -63,7 +63,14 @@ def natural_transitions(
         ]
     ]:
         """Returns the list of the `Processes` and `Events` in the `Problem`."""
-        return self.events + self.processes
+        ntlist: List[
+            Union[
+                up.model.natural_transition.Event, up.model.natural_transition.Process
+            ]
+        ] = []
+        ntlist.extend(self.processes)
+        ntlist.extend(self.events)
+        return ntlist
 
     def clear_events(self):
         """Removes all the `Problem` `Events`."""

diff --git a/unified_planning/test/test_pddl_io.py b/unified_planning/test/test_pddl_io.py
@@ -577,7 +577,6 @@ def test_examples_io(self):
                 w.write_problem(problem_filename)
 
                 reader = PDDLReader()
-                print(problem)
                 parsed_problem = reader.parse_problem(domain_filename, problem_filename)
 
                 # Case where the reader does not convert the final_value back to actions_cost.