refactoring: comments

unified_planning/io/pddl_writer.py

@@ -533,7 +533,7 @@ def _write_domain(self, out: IO[str]):
         )
         costs: Dict[
             Union[up.model.NaturalTransition, up.model.Action], Optional[up.model.FNode]
-        ] = {}  # TODO check if natural_transition should be here
+        ] = {}
         metrics = self.problem.quality_metrics
         if len(metrics) == 1:
             metric = metrics[0]

unified_planning/model/action.py

@@ -13,9 +13,7 @@
 # limitations under the License.
 #
 """
-This module defines the `Action` base class and some of his extensions.
-An `Action` has a `name`, a `list` of `Parameter`, a `list` of `preconditions`
-and a `list` of `effects`.
+This module defines the `Action` class and some of his extensions.
 """
 
 

unified_planning/model/natural_transition.py

@@ -29,17 +29,11 @@
 
 from unified_planning.model.transition import Transition
 
-# TODO fix comments here
-# TODO check weather events/processes should have all the methods/other stuff that actions have. If yes, we need more tests.
+# TODO check weather events/processes should have all the methods/other stuff that actions have.
+# If yes, we probably need more tests for better coverage
 
 """
-This module defines the `NaturalTransition` class and some of his extensions.
-An `NaturalTransition` has a `name`, a `list` of `Parameter`, a `list` of `preconditions`
-and a `list` of `effects`.
-"""
-
-"""
-Below we have natural transitions. These are not controlled by the agent and would probably need a proper subclass. Natural transitions can be of two kinds:
+Below we have natural transitions. These are not controlled by the agent. Natural transitions can be of two kinds:
 Processes or Events.
 Processes dictate how numeric variables evolve over time through the use of time-derivative functions
 Events dictate the analogous of urgent transitions in timed automata theory