From c32cf21b634ea28e848dc22b8346897ec0e06ae4 Mon Sep 17 00:00:00 2001 From: Daniel Agar Date: Mon, 24 Oct 2022 20:41:24 -0400 Subject: [PATCH] commander: estimator check shorten messages - otherwise these are awkwardly split in mavlink --- .../HealthAndArmingChecks/checks/estimatorCheck.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/modules/commander/HealthAndArmingChecks/checks/estimatorCheck.cpp b/src/modules/commander/HealthAndArmingChecks/checks/estimatorCheck.cpp index 5eb9d1bf8e..53a155feb5 100644 --- a/src/modules/commander/HealthAndArmingChecks/checks/estimatorCheck.cpp +++ b/src/modules/commander/HealthAndArmingChecks/checks/estimatorCheck.cpp @@ -147,10 +147,10 @@ void EstimatorChecks::checkEstimatorStatus(const Context &context, Report &repor */ reporter.armingCheckFailure(required_groups, health_component_t::local_position_estimate, events::ID("check_estimator_hor_vel_not_stable"), - events::Log::Error, "Horizontal velocity estimate not stable"); + events::Log::Error, "Horizontal velocity unstable"); if (reporter.mavlink_log_pub()) { - mavlink_log_critical(reporter.mavlink_log_pub(), "Preflight Fail: horizontal velocity estimate not stable"); + mavlink_log_critical(reporter.mavlink_log_pub(), "Preflight Fail: horizontal velocity unstable"); } } else if (!context.isArmed() && estimator_status.pre_flt_fail_innov_vel_vert) { @@ -158,10 +158,10 @@ void EstimatorChecks::checkEstimatorStatus(const Context &context, Report &repor */ reporter.armingCheckFailure(required_groups, health_component_t::local_position_estimate, events::ID("check_estimator_vert_vel_not_stable"), - events::Log::Error, "Vertical velocity estimate not stable"); + events::Log::Error, "Vertical velocity unstable"); if (reporter.mavlink_log_pub()) { - mavlink_log_critical(reporter.mavlink_log_pub(), "Preflight Fail: vertical velocity estimate not stable"); + mavlink_log_critical(reporter.mavlink_log_pub(), "Preflight Fail: vertical velocity unstable"); } } else if (!context.isArmed() && estimator_status.pre_flt_fail_innov_height) { @@ -194,10 +194,10 @@ void EstimatorChecks::checkEstimatorStatus(const Context &context, Report &repor */ reporter.armingCheckFailure(required_groups_mag, health_component_t::local_position_estimate, events::ID("check_estimator_mag_interference"), - events::Log::Warning, "Strong magnetic interference detected"); + events::Log::Warning, "Strong magnetic interference"); if (reporter.mavlink_log_pub()) { - mavlink_log_critical(reporter.mavlink_log_pub(), "Preflight Fail: Strong magnetic interference detected"); + mavlink_log_critical(reporter.mavlink_log_pub(), "Preflight Fail: Strong magnetic interference"); } }