ENH: Refactor parachute trigger function and improve controller observed variables retrieval

Author: MateusStano

docs/examples/camoes_flight_sim.ipynb

@@ -298,19 +298,28 @@ def __evaluate_trigger_function(self, trigger):
                 )
 
             def triggerfunc(p, h, y, sensors, **kwargs):
+                if not positional_param_count and not accepts_var_positional:
+                    # New-style kwargs-only trigger: no positional args accepted.
+                    # Restore sensors (removed by trigger_event_wrapper) and
+                    # override pressure with the noisy value before forwarding.
+                    merged = {**kwargs, "pressure": p, "sensors": sensors}
+                    if accepts_var_keyword:
+                        return trigger(**merged)
+                    return trigger(
+                        **{k: v for k, v in merged.items() if k in keyword_only_names}
+                    )
+
                 positional_args = [p, h, y]
                 if accepts_sensors_positional:
                     positional_args.append(sensors)
 
                 if accepts_var_keyword:
                     return trigger(*positional_args, **kwargs)
 
-                forwarded_kwargs = {
-                    key: value
-                    for key, value in kwargs.items()
-                    if key in keyword_only_names
-                }
-                return trigger(*positional_args, **forwarded_kwargs)
+                return trigger(
+                    *positional_args,
+                    **{k: v for k, v in kwargs.items() if k in keyword_only_names},
+                )
 
             self.triggerfunc = triggerfunc
 

rocketpy/rocket/parachute.py

@@ -1877,7 +1877,7 @@ def add_air_brakes(
                 "instead. Support for this argument will be removed in v1.13.",
                 DeprecationWarning,
             )
-            controller_context["observed_variables"] = initial_observed_variables.copy()
+            controller_context["observed_variables"] = initial_observed_variables
 
         orig_controller = controller_function
         signature = inspect.signature(orig_controller)

rocketpy/rocket/rocket.py

@@ -2723,7 +2723,23 @@ def from_dict(cls, data):
         alternative="Access the desired variables via controller.log",
     )
     def get_controller_observed_variables(self):
-        return None
+        """Retrieve the observed variables from each controller.
+
+        If there is only one controller, its log is returned directly. If
+        there are multiple controllers, a list of logs is returned.
+
+        Returns
+        -------
+        list
+            Controller log(s) containing the return values of each
+            controller function call.
+        """
+        observed_variables = [controller.log for controller in self._controllers]
+        return (
+            observed_variables[0]
+            if len(observed_variables) == 1
+            else observed_variables
+        )
 
     @deprecated(
         reason="Prefer direct pair iteration (for example zip(seq, seq[1:]))",