976 feature display a table of data in fla (#1048)

* Start adding FLA data table

* Fix FLA data table styling and message display

* Fix UTC time conversion

* Add correct utc time conversion to data table

* Address copilot review comments

* Fix merge

* Fix table and other messages for dataflash_log file

Author: 1Blademaster

gcs/electron/fla.ts

@@ -129,6 +129,7 @@ async function parseDataflashLogFile(
         // File data
       } else if (messageName === "MSG") {
         // MSG data
+        // TODO: Use "VER" message for this data
         const text = splitLineData[2]?.trim()
 
         if (aircraftType === null && text) {
@@ -139,43 +140,39 @@ async function parseDataflashLogFile(
             aircraftType = "copter"
           }
         }
-      } else {
-        // Message data
-        const formatMessage = formatMessages[messageName]
-        if (formatMessage) {
-          if (!messages[messageName]) {
-            messages[messageName] = []
-          }
+      }
 
-          const messageObj: MessageObject = {
-            name: messageName,
-          }
+      // Message data
+      const formatMessage = formatMessages[messageName]
+      if (formatMessage) {
+        if (!messages[messageName]) {
+          messages[messageName] = []
+        }
+
+        const messageObj: MessageObject = {
+          name: messageName,
+        }
 
-          const fields = formatMessage.fields
-          const format = formatMessage.format
-          const fieldsLength = fields.length
-
-          for (
-            let i = 0;
-            i < fieldsLength && i < splitLineData.length - 1;
-            i++
-          ) {
-            const field = fields[i]
-            const formatType = format[i]
-            const value = splitLineData[i + 1]?.trim()
-
-            if (value !== undefined && value !== "") {
-              if (stringTypes.has(formatType)) {
-                messageObj[field] = value
-              } else {
-                const numValue = parseFloat(value)
-                messageObj[field] = isNaN(numValue) ? 0 : numValue
-              }
+        const fields = formatMessage.fields
+        const format = formatMessage.format
+        const fieldsLength = fields.length
+
+        for (let i = 0; i < fieldsLength && i < splitLineData.length - 1; i++) {
+          const field = fields[i]
+          const formatType = format[i]
+          const value = splitLineData[i + 1]?.trim()
+
+          if (value !== undefined && value !== "") {
+            if (stringTypes.has(formatType)) {
+              messageObj[field] = value
+            } else {
+              const numValue = parseFloat(value)
+              messageObj[field] = isNaN(numValue) ? 0 : numValue
             }
           }
-
-          ;(messages[messageName] as MessageObject[]).push(messageObj)
         }
+
+        ;(messages[messageName] as MessageObject[]).push(messageObj)
       }
 
       const now = Date.now()
@@ -696,8 +693,12 @@ export async function getMessages(
 
     for (let j = 0; j < len; j++) {
       const point = series[j]
+      // Use UtcTimeUS if available, otherwise fall back to TimeUS
+      const timeValue =
+        point.UtcTimeUS !== undefined ? point.UtcTimeUS : point.TimeUS
+
       // making sure all the entries are numbers
-      x[j] = typeof point.TimeUS === "number" ? point.TimeUS : 0
+      x[j] = typeof timeValue === "number" ? timeValue : 0
       y[j] = typeof point[fieldName] === "number" ? point[fieldName] : 0
     }
 
@@ -712,6 +713,27 @@ export async function getMessages(
   return datasets
 }
 
+export async function getMessageDataForTable(
+  _event: unknown,
+  requestedMessage: string,
+): Promise<MessageObject[] | { success: false; error: string }> {
+  if (!logData) {
+    console.error("getMessageDataForTable: logData is null or undefined.")
+    return { success: false, error: "Log data not loaded." }
+  }
+
+  if (!requestedMessage.trim()) {
+    return { success: false, error: "Invalid message type." }
+  }
+
+  const series = logData[requestedMessage] as MessageObject[]
+  if (!Array.isArray(series) || series.length === 0) {
+    return { success: false, error: "No data available for this message type." }
+  }
+
+  return series
+}
+
 export async function saveParamsToFile(
   _event: unknown,
   filePath: string,

gcs/electron/main.ts

@@ -18,6 +18,7 @@ import packageInfo from "../package.json"
 
 import openFile, {
   clearRecentFiles,
+  getMessageDataForTable,
   getMessages,
   getRecentFiles,
   saveParamsToFile,
@@ -617,6 +618,9 @@ app.whenReady().then(() => {
   // Load Messages on demand
   ipcMain.handle("fla:get-messages", getMessages)
 
+  // Get message data for table
+  ipcMain.handle("fla:get-message-data-for-table", getMessageDataForTable)
+
   // Save FLA params to file
   ipcMain.handle("fla:save-params-to-file", saveParamsToFile)
 

gcs/electron/preload.js

@@ -7,6 +7,7 @@ const ALLOWED_INVOKE_CHANNELS = [
   "fla:get-recent-logs",
   "fla:clear-recent-logs",
   "fla:get-messages",
+  "fla:get-message-data-for-table",
   "fla:save-params-to-file",
   "app:get-save-file-path",
   "app:save-file",

gcs/electron/utils/flaUtils.ts

@@ -10,19 +10,13 @@ import type {
 } from "../types/flaTypes"
 
 const ignoredMessages: string[] = [
-  "ERR",
-  "EV",
-  "MSG",
-  "VER",
   "TIMESYNC",
-  "PARAM_VALUE",
   "TSYN",
   "UNIT",
   "FILE",
   "FMTU",
   "FMT",
   "MULT",
-  "PARM",
   "units",
   "format",
   "aircraftType",
@@ -240,7 +234,7 @@ export function convertTimeUStoUTC(
   logMessages: LoadedLogMessages,
   gpsOffset: number,
 ): LoadedLogMessages {
-  // This still takes some time for some reason
+  // Add UtcTimeUS alongside TimeUS instead of replacing it
   const convertedMessages: LoadedLogMessages = { ...logMessages }
 
   Object.keys(convertedMessages)
@@ -250,7 +244,7 @@ export function convertTimeUStoUTC(
       if (Array.isArray(messages)) {
         convertedMessages[key] = messages.map((message: MessageObject) => ({
           ...message,
-          TimeUS: (message.TimeUS as number) / 1000 + gpsOffset,
+          UtcTimeUS: (message.TimeUS as number) / 1000 + gpsOffset,
         }))
       }
     })

gcs/package.json

@@ -38,6 +38,7 @@
     "@robloche/chartjs-plugin-streaming": "^3.1.0",
     "@tabler/icons-react": "^3.37.1",
     "@tailwindcss/container-queries": "^0.1.1",
+    "@tanstack/react-virtual": "^3.13.19",
     "@tiptap/extension-link": "^2.11.6",
     "@tiptap/pm": "^2.11.6",
     "@tiptap/react": "^2.11.6",