Skip to content

Commit ccf4afa

Browse files
committed
🐛 fix reaction time for none managing
1 parent 3a04c4a commit ccf4afa

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

suave_metrics/suave_metrics/mission_metrics.py

+12-8
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ def maintain_motion_transition_cb(self, msg):
283283
self.component_recovery_time.append(reaction_time)
284284
self.thrusters_failed = False
285285
self.get_logger().info(
286-
'Component recovery time: {} seconds'.format(reaction_time))
286+
'Thruster failure reaction time: {} seconds'.format(reaction_time))
287287

288288
def generate_recharge_path_transition_cb(self, msg):
289289
if msg.goal_state.label == "active" and self.battery_low is True:
@@ -292,7 +292,7 @@ def generate_recharge_path_transition_cb(self, msg):
292292
self.battery_reaction_time.append(reaction_time)
293293
# self.battery_low = False
294294
self.get_logger().info(
295-
'Battery reaction time: {} seconds'.format(reaction_time))
295+
'Battery drop reaction time: {} seconds'.format(reaction_time))
296296

297297
def param_change_cb(self, msg):
298298
time = self.get_clock().now()
@@ -308,8 +308,7 @@ def param_change_cb(self, msg):
308308
reaction_time)
309309
self.wrong_altitude = False
310310
self.get_logger().info(
311-
'Water visibility correction time: {} seconds'.
312-
format(reaction_time))
311+
'Water visibility change reaction time: {0} seconds'.format(reaction_time))
313312
return
314313

315314
def save_mission_results_cb(
@@ -356,6 +355,14 @@ def save_mission_results(self) -> None:
356355
]
357356

358357
date = datetime.now().strftime("%d-%b-%Y-%H-%M-%S")
358+
mean_reaction_time = 0.0
359+
try:
360+
mean_reaction_time = statistics.fmean(
361+
self.component_recovery_time +
362+
self.wv_reaction_time +
363+
self.battery_reaction_time)
364+
except statistics.StatisticsError as e:
365+
pass
359366
mission_data = [
360367
self.mission_name,
361368
date,
@@ -365,10 +372,7 @@ def save_mission_results(self) -> None:
365372
pipeline_detected,
366373
detection_time_delta,
367374
self.distance_inspected,
368-
statistics.fmean(
369-
self.component_recovery_time +
370-
self.wv_reaction_time +
371-
self.battery_reaction_time)
375+
mean_reaction_time
372376
]
373377

374378
self.save_metrics(

0 commit comments

Comments
 (0)