From d5eab23f5361971cac2b25f1c647f6e0895c18d9 Mon Sep 17 00:00:00 2001 From: Jan Liphardt Date: Wed, 28 Jun 2017 00:28:33 -0400 Subject: [PATCH] airspeed error message clarification (#7497) In day to day practice, there are two reasons for fabsf(airspeed.differential_pressure_filtered_pa) > 15.0f 1/ The sensor has not been calibrated. In this case, the sensor needs to be calibrated. 2/ It's a windy day, and the operator has powered up the system without covering the pitot tube. In that case, you need to reboot with the pitot cover in place. The original message ('Check calibration') is ok, but why not be more direct? The user does not have to "check the calibration" - the user has to (a) either calibrate the system in the first place, or (b) not forget the pitot tube cover when powering up the system. --- src/modules/commander/PreflightCheck.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/modules/commander/PreflightCheck.cpp b/src/modules/commander/PreflightCheck.cpp index c955f9cef2..97824ca3f9 100644 --- a/src/modules/commander/PreflightCheck.cpp +++ b/src/modules/commander/PreflightCheck.cpp @@ -411,7 +411,7 @@ static bool airspeedCheck(orb_advert_t *mavlink_log_pub, bool optional, bool rep if (fabsf(airspeed.differential_pressure_filtered_pa) > 15.0f && !prearm) { if (report_fail) { - mavlink_log_critical(mavlink_log_pub, "PREFLIGHT FAIL: HIGH AIRSPEED, CHECK CALIBRATION"); + mavlink_log_critical(mavlink_log_pub, "PREFLIGHT FAIL: AIRSPEED CALIBRATION BAD OR PITOT UNCOVERED"); } success = false; goto out;