Fix IMU sensor refresh rate selection (betaflight#3642)

* Fix sensor refresh rate selection * Fix html * Refactor
chmelevskij · Apr 27, 2024 · b53f200 · b53f200
1 parent a0f8e90
commit b53f200
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/src/js/tabs/sensors.js b/src/js/tabs/sensors.js
@@ -346,14 +346,25 @@ sensors.initialize = function (callback) {
             const scales = {
                 'gyro':  parseFloat($('.tab-sensors select[name="gyro_scale"]').val()),
                 'accel': parseFloat($('.tab-sensors select[name="accel_scale"]').val()),
-                'mag':   parseFloat($('.tab-sensors select[name="mag_scale"]').val()),
+                'mag':   parseInt($('.tab-sensors select[name="mag_scale"]').val(), 10),
             };
 
             // handling of "data pulling" is a little bit funky here, as MSP_RAW_IMU contains values for gyro/accel/mag but not altitude
             // this means that setting a slower refresh rate on any of the attributes would have no effect
             // what we will do instead is = determinate the fastest refresh rate for those 3 attributes, use that as a "polling rate"
             // and use the "slower" refresh rates only for re-drawing the graphs (to save resources/computing power)
-            const fastest = d3.min([rates.gyro, rates.accel, rates.mag]);
+
+            let fastest;
+            // if any of the refresh rates change, we need to re-determine the fastest refresh rate
+            if (['gyro_refresh_rate', 'accel_refresh_rate', 'mag_refresh_rate'].includes($(this).attr('name'))) {
+                fastest = $(this).val();
+
+                $('.tab-sensors select[name="gyro_refresh_rate"]').val(fastest);
+                $('.tab-sensors select[name="accel_refresh_rate"]').val(fastest);
+                $('.tab-sensors select[name="mag_refresh_rate"]').val(fastest);
+            } else {
+                fastest = d3.max([rates.gyro, rates.accel, rates.mag]);
+            }
 
             // store current/latest refresh rates in the storage
             setConfig({'sensor_settings': {'rates': rates, 'scales': scales}});

diff --git a/src/tabs/sensors.html b/src/tabs/sensors.html
@@ -128,7 +128,7 @@
                     <dl>
                         <dt i18n="sensorsRefresh"></dt>
                         <dd class="rate">
-                            <select name="mag_refrash_rate">
+                            <select name="mag_refresh_rate">
                                 <option value="10">10 ms</option>
                                 <option value="20">20 ms</option>
                                 <option value="30">30 ms</option>