859 dispalay gps hdop on dashboard (#864)

* added hdop to setGPSRawintData in droneInfo slice

* Added hdop logic to middleware. The difference between hdop and eph confused me for a bit so added in a check incase we ever are in a situation where hdop is sent from mavlink instead of eph

* front end hdop finished

* spelling error

* more spelling errors

* changed status bar condition from hdop != 0 to hdop > 0 as per copilot reccomendation

* linter fix :)

* copilot fix

* added hdop to setGPSRawintData in droneInfo slice

* Added hdop logic to middleware. The difference between hdop and eph confused me for a bit so added in a check incase we ever are in a situation where hdop is sent from mavlink instead of eph

* front end hdop finished

* spelling error

* more spelling errors

* changed status bar condition from hdop != 0 to hdop > 0 as per copilot reccomendation

* linter fix :)

* copilot fix

* removed check from .hdop presense in middlware

* made HDOP always visible on stausbar

* switched to IconTarget icon

* moved hdop section across in status bar to between lat/lon and gps

* changed hdop to display no gps instead of 0.00

* linters :)

* changed HDOP to display 0.00 when no gps available instead of "NO GPS"

* Removed "precision" from HDoP tooltip

Author: Matthew-Dobson

gcs/src/components/dashboard/statusBar.jsx

@@ -136,6 +136,7 @@ export default function StatusBar(props) {
         style={{ backgroundColor: GetOutsideVisibilityColor() }}
       >
         {props.children}
+
         <StatusSection
           icon={isConnectedToSocket ? <IconNetwork /> : <IconNetworkOff />}
           value=""

gcs/src/dashboard.jsx

@@ -22,6 +22,7 @@ import {
   IconGps,
   IconRadar,
   IconSatellite,
+  IconTarget,
 } from "@tabler/icons-react"
 import { ResizableBox } from "react-resizable"
 
@@ -75,7 +76,10 @@ export default function Dashboard() {
   const batteryData = useSelector(selectBatteryData)
   const statustextMessages = useSelector(selectMessages)
   const armedNotification = useSelector(selectNotificationSound)
-  const { fixType, satellitesVisible } = useSelector(selectGPSRawInt)
+  const { fixType, satellitesVisible, hdop } = useSelector(selectGPSRawInt)
+
+  const hdopDisplay = hdop != null ? hdop.toFixed(2) : "0.00"
+
   const connectedToDrone = useSelector(selectConnectedToDrone)
 
   // Telemetry panel sizing
@@ -197,6 +201,11 @@ export default function Dashboard() {
             value={GPS_FIX_TYPES[fixType]}
             tooltip="GPS fix type"
           />
+          <StatusSection
+            icon={<IconTarget />}
+            value={hdopDisplay}
+            tooltip="GPS HDoP"
+          />
           <StatusSection
             icon={<IconGps />}
             value={`(${lat !== undefined ? lat.toFixed(7) : 0}, ${

gcs/src/redux/middleware/socketMiddleware.js

@@ -214,10 +214,19 @@ const socketMiddleware = (store) => {
           setOnboardControlSensorsEnabled(msg.onboard_control_sensors_enabled),
         )
         break
-      case "GPS_RAW_INT":
-        store.dispatch(setGpsRawIntData(msg))
+      case "GPS_RAW_INT": {
+        // MAVLink GPS_RAW_INT provides 'eph' (HDOP * 100).
+        const hdop = msg.eph != null ? msg.eph / 100.0 : null
+
+        store.dispatch(
+          setGpsRawIntData({
+            ...msg,
+            hdop,
+          }),
+        )
         store.dispatch(calculateGpsTrackHeadingThunk())
         break
+      }
       case "RC_CHANNELS":
         // NOTE: UNABLE TO TEST IN SIMULATOR!
         store.dispatch(setRSSIData(msg.rssi))

gcs/src/redux/slices/droneInfoSlice.js

@@ -57,6 +57,7 @@ const droneInfoSlice = createSlice({
       satellitesVisible: 0,
       velocity: 0,
       courseOverGround: 0,
+      hdop: 0,
     },
     rssi: 0.0,
     notificationSound: "",
@@ -179,6 +180,7 @@ const droneInfoSlice = createSlice({
         state.gpsRawIntData.fixType = action.payload.fix_type
         state.gpsRawIntData.velocity = action.payload.vel / 100.0 // cm/s to m/s
         state.gpsRawIntData.courseOverGround = centiDegToDeg(action.payload.cog)
+        state.gpsRawIntData.hdop = action.payload.hdop ?? 0
       }
     },
     setOnboardControlSensorsEnabled: (state, action) => {