diff --git a/src/modules/commander/HealthAndArmingChecks/checks/armPermissionCheck.cpp b/src/modules/commander/HealthAndArmingChecks/checks/armPermissionCheck.cpp
index d25cff98d5..528935b8e8 100644
--- a/src/modules/commander/HealthAndArmingChecks/checks/armPermissionCheck.cpp
+++ b/src/modules/commander/HealthAndArmingChecks/checks/armPermissionCheck.cpp
@@ -37,13 +37,13 @@ void ArmPermissionChecks::checkAndReport(const Context &context, Report &reporte
{
if (_param_com_armable.get() < 1) {
/* EVENT
- * @description
- * Vehicle is in safety configuration and denies arming.
- *
- *
- * This check can be configured via COM_ARMABLE parameter.
- *
- */
+ * @description
+ * Vehicle is in safety configuration and denies arming.
+ *
+ *
+ * This check can be configured via COM_ARMABLE parameter.
+ *
+ */
reporter.armingCheckFailure(NavModes::All, health_component_t::system,
events::ID("check_armable_configuration"),
events::Log::Error, "Vehicle is in safety configuration");
diff --git a/src/modules/commander/HealthAndArmingChecks/checks/batteryCheck.cpp b/src/modules/commander/HealthAndArmingChecks/checks/batteryCheck.cpp
index 9b40fb39ac..1d6fb818c0 100644
--- a/src/modules/commander/HealthAndArmingChecks/checks/batteryCheck.cpp
+++ b/src/modules/commander/HealthAndArmingChecks/checks/batteryCheck.cpp
@@ -211,13 +211,13 @@ void BatteryChecks::checkAndReport(const Context &context, Report &reporter)
// This is declared critical so QGC displays a yellow box and reads "low battery" out loud making the user aware
/* EVENT
- * @description
- * The lowest battery state of charge is below the low threshold.
- *
- *
- * Can be configured with BAT_LOW_THR.
- *
- */
+ * @description
+ * The lowest battery state of charge is below the low threshold.
+ *
+ *
+ * Can be configured with BAT_LOW_THR.
+ *
+ */
reporter.armingCheckFailure(affected_modes, health_component_t::battery, events::ID("check_battery_low"),
events::Log::Critical, "Low battery");
@@ -229,13 +229,13 @@ void BatteryChecks::checkAndReport(const Context &context, Report &reporter)
case battery_status_s::WARNING_CRITICAL:
/* EVENT
- * @description
- * The lowest battery state of charge is below the critical threshold.
- *
- *
- * Can be configured with BAT_CRIT_THR and from when to disalow arming with COM_ARM_BAT_MIN.
- *
- */
+ * @description
+ * The lowest battery state of charge is below the critical threshold.
+ *
+ *
+ * Can be configured with BAT_CRIT_THR and from when to disalow arming with COM_ARM_BAT_MIN.
+ *
+ */
reporter.armingCheckFailure(affected_modes, health_component_t::battery, events::ID("check_battery_critical"),
events::Log::Critical, "Critical battery");
@@ -247,13 +247,13 @@ void BatteryChecks::checkAndReport(const Context &context, Report &reporter)
case battery_status_s::WARNING_EMERGENCY:
/* EVENT
- * @description
- * The lowest battery state of charge is below the emergency threshold.
- *
- *
- * Can be configured with BAT_EMERGEN_THR.
- *
- */
+ * @description
+ * The lowest battery state of charge is below the emergency threshold.
+ *
+ *
+ * Can be configured with BAT_EMERGEN_THR.
+ *
+ */
reporter.armingCheckFailure(affected_modes, health_component_t::battery, events::ID("check_battery_emergency"),
events::Log::Emergency, "Emergency battery level");
diff --git a/src/modules/commander/HealthAndArmingChecks/checks/escCheck.cpp b/src/modules/commander/HealthAndArmingChecks/checks/escCheck.cpp
index e78428242a..c64a5b2877 100644
--- a/src/modules/commander/HealthAndArmingChecks/checks/escCheck.cpp
+++ b/src/modules/commander/HealthAndArmingChecks/checks/escCheck.cpp
@@ -140,11 +140,11 @@ uint16_t EscChecks::checkEscOnline(const Context &context, Report &reporter, con
uint8_t esc_nr = esc_index + 1;
/* EVENT
- * @description
- *
- * This check can be configured via COM_ARM_CHK_ESCS parameter.
- *
- */
+ * @description
+ *
+ * This check can be configured via COM_ARM_CHK_ESCS parameter.
+ *
+ */
reporter.healthFailure(NavModes::All, health_component_t::motors_escs, events::ID("check_escs_offline"),
events::Log::Critical, "ESC {1} offline", esc_nr);
snprintf(esc_fail_msg + strlen(esc_fail_msg), sizeof(esc_fail_msg) - strlen(esc_fail_msg), "ESC%d ", esc_nr);
@@ -206,13 +206,13 @@ uint16_t EscChecks::checkEscStatus(const Context &context, Report &reporter, con
}
/* EVENT
- * @description
- * {3}
- *
- *
- * This check can be configured via COM_ARM_CHK_ESCS parameter.
- *
- */
+ * @description
+ * {3}
+ *
+ *
+ * This check can be configured via COM_ARM_CHK_ESCS parameter.
+ *
+ */
reporter.healthFailure(
NavModes::All, health_component_t::motors_escs, events::ID("check_failure_detector_arm_esc"),
events::Log::Critical, "ESC {1}: {2}", esc_index + 1, fault_reason_index, action);
@@ -291,11 +291,11 @@ uint16_t EscChecks::checkMotorStatus(const Context &context, Report &reporter, c
if (_esc_undercurrent_hysteresis[i].get_state()) {
/* EVENT
- * @description
- *
- * This check can be configured via FD_ACT_EN parameter.
- *
- */
+ * @description
+ *
+ * This check can be configured via FD_ACT_EN parameter.
+ *
+ */
reporter.healthFailure(NavModes::All, health_component_t::motors_escs,
events::ID("check_failure_detector_motor_uc"),
events::Log::Critical, "Motor {1} undercurrent detected", actuator_function_index + 1);
@@ -308,11 +308,11 @@ uint16_t EscChecks::checkMotorStatus(const Context &context, Report &reporter, c
if (_esc_overcurrent_hysteresis[i].get_state()) {
/* EVENT
- * @description
- *
- * This check can be configured via FD_ACT_EN parameter.
- *
- */
+ * @description
+ *
+ * This check can be configured via FD_ACT_EN parameter.
+ *
+ */
reporter.healthFailure(NavModes::All, health_component_t::motors_escs,
events::ID("check_failure_detector_motor_oc"),
events::Log::Critical, "Motor {1} overcurrent detected", actuator_function_index + 1);
@@ -352,11 +352,11 @@ void EscChecks::updateEscsStatus(const Context &context, Report &reporter, const
if (_esc_arm_hysteresis.get_state()) {
/* EVENT
- * @description
- *
- * This check can be configured via COM_ARM_CHK_ESCS parameter.
- *
- */
+ * @description
+ *
+ * This check can be configured via COM_ARM_CHK_ESCS parameter.
+ *
+ */
reporter.healthFailure(NavModes::All, health_component_t::motors_escs,
events::ID("check_escs_not_all_armed"),
events::Log::Critical, "Not all ESCs are armed");
diff --git a/src/modules/commander/HealthAndArmingChecks/checks/flightTimeCheck.cpp b/src/modules/commander/HealthAndArmingChecks/checks/flightTimeCheck.cpp
index adb8a02d7e..32aea019eb 100644
--- a/src/modules/commander/HealthAndArmingChecks/checks/flightTimeCheck.cpp
+++ b/src/modules/commander/HealthAndArmingChecks/checks/flightTimeCheck.cpp
@@ -80,9 +80,9 @@ void FlightTimeChecks::checkAndReport(const Context &context, Report &reporter)
}
/* EVENT
- * @description
- * Maximal flight time warning (less than 1min remaining)
- */
+ * @description
+ * Maximal flight time warning (less than 1min remaining)
+ */
events::send(events::ID("commander_max_flight_time_warning_seconds"), events::Log::Warning,
"Approaching max flight time (system will RTL in {1} seconds)", floored_remaining_flight_time_sec);
@@ -97,9 +97,9 @@ void FlightTimeChecks::checkAndReport(const Context &context, Report &reporter)
}
/* EVENT
- * @description
- * Maximal flight time warning (more than 1min remaining)
- */
+ * @description
+ * Maximal flight time warning (more than 1min remaining)
+ */
events::send(events::ID("commander_max_flight_time_warning_minutes"), events::Log::Warning,
"Approaching max flight time (system will RTL in {1} minutes)", floored_remaining_flight_time_min);
}
diff --git a/src/modules/commander/HealthAndArmingChecks/checks/geofenceCheck.cpp b/src/modules/commander/HealthAndArmingChecks/checks/geofenceCheck.cpp
index ceb8458721..f3c34097ee 100644
--- a/src/modules/commander/HealthAndArmingChecks/checks/geofenceCheck.cpp
+++ b/src/modules/commander/HealthAndArmingChecks/checks/geofenceCheck.cpp
@@ -51,11 +51,11 @@ void GeofenceChecks::checkAndReport(const Context &context, Report &reporter)
if (geofence_result.geofence_max_dist_triggered) {
/* EVENT
- * @description
- *
- * This check can be configured via GF_ACTION and GF_MAX_HOR_DIST parameters.
- *
- */
+ * @description
+ *
+ * This check can be configured via GF_ACTION and GF_MAX_HOR_DIST parameters.
+ *
+ */
reporter.armingCheckFailure(NavModes::All, health_component_t::system,
events::ID("check_gf_violation_max_hor_dist"),
events::Log::Error, "Geofence violation: exceeding maximum distance to Home");
@@ -67,11 +67,11 @@ void GeofenceChecks::checkAndReport(const Context &context, Report &reporter)
if (geofence_result.geofence_max_alt_triggered) {
/* EVENT
- * @description
- *
- * This check can be configured via GF_ACTION and GF_MAX_VER_DIST parameters.
- *
- */
+ * @description
+ *
+ * This check can be configured via GF_ACTION and GF_MAX_VER_DIST parameters.
+ *
+ */
reporter.armingCheckFailure(NavModes::All, health_component_t::system,
events::ID("check_gf_violation_max_alt"),
events::Log::Error, "Geofence violation: exceeding maximum altitude above Home");
@@ -83,11 +83,11 @@ void GeofenceChecks::checkAndReport(const Context &context, Report &reporter)
if (geofence_result.geofence_custom_fence_triggered) {
/* EVENT
- * @description
- *
- * This check can be configured via GF_ACTION parameter.
- *
- */
+ * @description
+ *
+ * This check can be configured via GF_ACTION parameter.
+ *
+ */
reporter.armingCheckFailure(NavModes::All, health_component_t::system,
events::ID("check_gf_violation_custom_gf"),
events::Log::Error, "Geofence violation: approaching or outside geofence");