From e2a6770d1d8cdf299ab2dd0bbe965b96c99e3746 Mon Sep 17 00:00:00 2001
From: j-w9 <295470691+j-w9@users.noreply.github.com>
Date: Sat, 24 Oct 2026 12:13:06 -0400
Subject: [PATCH 1/2] FilterReview: warn if raw IMU data is missing for much of
 the analysis time

---
 FilterReview/FilterReview.js | 43 ++++++++++++++++++++++++++++++++++++
 FilterReview/index.html      |  3 +++
 2 files changed, 46 insertions(+)

diff --git a/FilterReview/FilterReview.js b/FilterReview/FilterReview.js
index 9f0bdea5..e2ab6104 100644
--- a/FilterReview/FilterReview.js
+++ b/FilterReview/FilterReview.js
@@ -274,6 +274,42 @@ function HarmonicNotchFilter(params) {
     return this
 }
 
+// Warn if raw data is missing for a large part of the analysis time
+function update_missing_data_warning() {
+    for (let i = 0; i < 3; i++) {
+        let missing = 0
+        if (Gyro_batch.type == "raw") {
+            for (let j = 0; j < Gyro_batch.length; j++) {
+                if ((Gyro_batch[j] == null) || (Gyro_batch[j].sensor_num != i)) {
+                    continue
+                }
+                missing = Math.max(missing, get_missing_fraction(Gyro_batch[j]))
+            }
+        }
+        let missing_label = document.getElementById("Gyro" + i + "_missing")
+        missing_label.hidden = missing < missing_data_warn_fraction
+        missing_label.lastElementChild.textContent = "Missing data: " + Math.round(missing * 100) + "% of analysis time"
+    }
+}
+
+// Fraction of analysis time that has no raw data, batches are split where data is missing
+const missing_data_warn_fraction = 0.1
+function get_missing_fraction(batches) {
+    const first = batches[0]
+    const last = batches[batches.length - 1]
+    const start = Math.max(parseFloat(document.getElementById("TimeStart").value), first.sample_time)
+    const end = Math.min(parseFloat(document.getElementById("TimeEnd").value), last.sample_time + (last.x.length / last.sample_rate))
+    if (!(end > start)) {
+        return 0
+    }
+    let covered = 0
+    for (const batch of batches) {
+        const batch_end = batch.sample_time + (batch.x.length / batch.sample_rate)
+        covered += Math.max(0, Math.min(end, batch_end) - Math.max(start, batch.sample_time))
+    }
+    return Math.max(0, 1 - (covered / (end - start)))
+}
+
 function run_batch_fft(data_set) {
     const num_batch = data_set.length
     const num_points = data_set[0].x.length
@@ -378,6 +414,9 @@ function reset() {
     document.getElementById("filter_version_1").checked = true
     document.getElementById("calculate").disabled = true
     document.getElementById("calculate_filters").disabled = true
+    for (let i = 0; i < 3; i++) {
+        document.getElementById("Gyro" + i + "_missing").hidden = true
+    }
     document.getElementById("OpenFilterTool").disabled = true
     document.getElementById("SaveParams").disabled = true
     document.getElementById("LoadParams").disabled = true
@@ -763,6 +802,8 @@ function re_calc() {
 
     calculate()
 
+    update_missing_data_warning()
+
     load_filters()
 
     calculate_transfer_function()
@@ -2690,6 +2731,8 @@ async function load(log_file) {
     // Calculate FFT
     calculate()
 
+    update_missing_data_warning()
+
     // Update transfer function from filter setting
     calculate_transfer_function()
 
diff --git a/FilterReview/index.html b/FilterReview/index.html
index caec117a..986a62b0 100644
--- a/FilterReview/index.html
+++ b/FilterReview/index.html
@@ -231,6 +231,7 @@ <h2 style="text-align:center">IMU Spectrum
                 </p>
                 Logging rate: <label id="Gyro0_FFT_infoA"></label> Hz<br><br>
                 Frequency resolution: <label id="Gyro0_FFT_infoB"></label> Hz
+                <span id="Gyro0_missing" style="color:#c00000" hidden><br><br><span></span></span>
              </fieldset>
         </td>
         <td>
@@ -284,6 +285,7 @@ <h2 style="text-align:center">IMU Spectrum
                 </p>
                 Logging rate: <label id="Gyro1_FFT_infoA"></label> Hz<br><br>
                 Frequency resolution: <label id="Gyro1_FFT_infoB"></label> Hz
+                <span id="Gyro1_missing" style="color:#c00000" hidden><br><br><span></span></span>
             </fieldset>
         </td>
         <td>
@@ -337,6 +339,7 @@ <h2 style="text-align:center">IMU Spectrum
                 </p>
                 Logging rate: <label id="Gyro2_FFT_infoA"></label> Hz<br><br>
                 Frequency resolution: <label id="Gyro2_FFT_infoB"></label> Hz
+                <span id="Gyro2_missing" style="color:#c00000" hidden><br><br><span></span></span>
             </fieldset>
         </td>
     </tr>

From 2a4ff1588726e7a9b5bfd202672c96e300d7939c Mon Sep 17 00:00:00 2001
From: j-w9 <295470691+j-w9@users.noreply.github.com>
Date: Sat, 24 Oct 2026 12:13:40 -0400
Subject: [PATCH 2/2] FilterReview: move missing data reset into per gyro loop

---
 FilterReview/FilterReview.js | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/FilterReview/FilterReview.js b/FilterReview/FilterReview.js
index e2ab6104..5462d257 100644
--- a/FilterReview/FilterReview.js
+++ b/FilterReview/FilterReview.js
@@ -414,9 +414,6 @@ function reset() {
     document.getElementById("filter_version_1").checked = true
     document.getElementById("calculate").disabled = true
     document.getElementById("calculate_filters").disabled = true
-    for (let i = 0; i < 3; i++) {
-        document.getElementById("Gyro" + i + "_missing").hidden = true
-    }
     document.getElementById("OpenFilterTool").disabled = true
     document.getElementById("SaveParams").disabled = true
     document.getElementById("LoadParams").disabled = true
@@ -433,6 +430,7 @@ function reset() {
         }
         document.getElementById("SpecGyroInst" + i).disabled = true
         document.getElementById("BodeGyroInst" + i).disabled = true
+        document.getElementById("Gyro" + i + "_missing").hidden = true
 
         let fieldset = document.getElementById("Gyro" + i)
         fieldset.style["border-color"] = "rgb(192, 192, 192)"