From 6ef0046a1da5d2d994291494bf1e855ccff84eaf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?K=C3=A1roly=20Kiripolszky?= <karcsi@ekezet.com>
Date: Mon, 13 Mar 2023 16:02:09 +0100
Subject: [PATCH] Display average RPM on motor tab (#3371)

---
 src/js/tabs/motors.js | 29 ++++++++++++++++++++---------
 src/tabs/motors.html  |  4 ++--
 2 files changed, 22 insertions(+), 11 deletions(-)

diff --git a/src/js/tabs/motors.js b/src/js/tabs/motors.js
index 56de21a0..2164e08c 100644
--- a/src/js/tabs/motors.js
+++ b/src/js/tabs/motors.js
@@ -257,11 +257,6 @@ motors.initialize = async function (callback) {
 
         if (semver.lt(FC.CONFIG.apiVersion, API_VERSION_1_42) || !(FC.MOTOR_CONFIG.use_dshot_telemetry || FC.MOTOR_CONFIG.use_esc_sensor)) {
             $(".motor_testing .telemetry").hide();
-        } else {
-            // Hide telemetry from unused motors (to hide the tooltip in an empty blank space)
-            for (let i = FC.MOTOR_CONFIG.motor_count; i < FC.MOTOR_DATA.length; i++) {
-                $(`.motor_testing .telemetry .motor-${i}`).hide();
-            }
         }
 
         function setContentButtons(motorsTesting=false) {
@@ -1062,6 +1057,11 @@ motors.initialize = async function (callback) {
             const previousArmState = self.armed;
             const blockHeight = $('div.m-block:first').height();
             const motorValues = getMotorOutputs();
+            const MAX_VALUE_SIZE = 6,
+                AVG_RPM_ROUNDING = 100;
+            let sumRpm = 0,
+                isAllMotorValueEqual = motorValues.every((value, _index, arr) => value === arr[0]),
+                hasTelemetryError = false;
 
             for (let i = 0; i < motorValues.length; i++) {
                 const motorValue = motorValues[i];
@@ -1079,8 +1079,7 @@ motors.initialize = async function (callback) {
 
                 if (i < FC.MOTOR_CONFIG.motor_count && (FC.MOTOR_CONFIG.use_dshot_telemetry || FC.MOTOR_CONFIG.use_esc_sensor)) {
 
-                    const MAX_INVALID_PERCENT = 100,
-                          MAX_VALUE_SIZE = 6;
+                    const MAX_INVALID_PERCENT = 100;
 
                     let rpmMotorValue = FC.MOTOR_TELEMETRY_DATA.rpm[i];
 
@@ -1088,14 +1087,17 @@ motors.initialize = async function (callback) {
                     if (rpmMotorValue > 999999) {
                         rpmMotorValue = `${(rpmMotorValue / 1000000).toFixed(5 - (rpmMotorValue / 1000000).toFixed(0).toString().length)}M`;
                     }
-
+                    if (isAllMotorValueEqual) {
+                        sumRpm += Math.round(rpmMotorValue * AVG_RPM_ROUNDING) / AVG_RPM_ROUNDING;
+                    }
                     rpmMotorValue = rpmMotorValue.toString().padStart(MAX_VALUE_SIZE);
                     let telemetryText = i18n.getMessage('motorsRPM', {motorsRpmValue: rpmMotorValue});
 
                     if (FC.MOTOR_CONFIG.use_dshot_telemetry) {
 
                         let invalidPercent = FC.MOTOR_TELEMETRY_DATA.invalidPercent[i];
-                        let classError = (invalidPercent > MAX_INVALID_PERCENT) ? "warning" : "";
+                        hasTelemetryError = invalidPercent > MAX_INVALID_PERCENT;
+                        let classError = hasTelemetryError ? "warning" : "";
                         invalidPercent = (invalidPercent / 100).toFixed(2).toString().padStart(MAX_VALUE_SIZE);
 
                         telemetryText += `<br><span class="${classError}">`;
@@ -1116,6 +1118,15 @@ motors.initialize = async function (callback) {
                 }
             }
 
+            if (FC.MOTOR_CONFIG.use_dshot_telemetry && !hasTelemetryError && isAllMotorValueEqual) {
+                const avgRpm = (Math.round(sumRpm / motorValues.length * AVG_RPM_ROUNDING) / AVG_RPM_ROUNDING).toFixed(0),
+                    avgRpmMotorValue = avgRpm.toString().padStart(MAX_VALUE_SIZE),
+                    message = i18n.getMessage('motorsRPM', { motorsRpmValue: avgRpmMotorValue });
+                $(`.motor_testing .telemetry .motor-master`).html(message);
+            } else {
+                $(`.motor_testing .telemetry .motor-master`).html("");
+            }
+
             //keep the following here so at least we get a visual cue of our motor setup
             update_arm_status();
 
diff --git a/src/tabs/motors.html b/src/tabs/motors.html
index 3f432e05..d17fc680 100644
--- a/src/tabs/motors.html
+++ b/src/tabs/motors.html
@@ -235,7 +235,7 @@
                                             </select>
                                         </div>
                                     </div>
-                
+
                                     <div class="row">
                                         <div class="left-cell">
                                             <div i18n="sensorsScale"></div>
@@ -319,7 +319,7 @@
                                 <li><span class="motor-5 cf_tip" i18n_title="motorsTelemetryHelp">&nbsp;</span></li>
                                 <li><span class="motor-6 cf_tip" i18n_title="motorsTelemetryHelp">&nbsp;</span></li>
                                 <li><span class="motor-7 cf_tip" i18n_title="motorsTelemetryHelp">&nbsp;</span></li>
-                                <li>&nbsp;</li>
+                                <li><span class="motor-master cf_tip" i18n_title="motorsTelemetryHelp">&nbsp;</span></li>
                             </ul>
                         </div>
                         <div class="sliders">