Merged in f5soh/librepilot/LP-574_PFD_Battery_display (pull request #489)

LP-574 Check if adc inputs are configured before displaying battery data on PFD

Approved-by: Lalanne Laurent <f5soh@free.fr>
Approved-by: Philippe Renon <philippe_renon@yahoo.fr>

Author: f5soh

ground/gcs/src/share/qml/js/uav.js

@@ -318,6 +318,18 @@ function batteryModuleEnabled() {
     return (hwSettings.optionalModulesBattery == HwSettings.OptionalModules.Enabled);
 }
 
+function batteryADCConfigured() {
+    for (var i = 0; i < 8; i++) {
+        var adcRouting = hwSettings.getADCRouting(i);
+        if (adcRouting == HwSettings.ADCRouting.BatteryVoltage) {
+            return true;
+        } else if (adcRouting == HwSettings.ADCRouting.BatteryCurrent) {
+            return true;
+        }
+    }
+    return false;
+}
+
 function batteryNbCells() {
     return flightBatterySettings.nbCells;
 }

ground/gcs/src/share/qml/pfd/Info.qml

@@ -271,7 +271,7 @@ Item {
     }
 
     // Battery Info (Top)
-    // Only visible when PathPlan not active and Battery module enabled
+    // Only visible when PathPlan not active and Battery module enabled and ADC input configured
 
     SvgElementPositionItem {
         id: topbattery_voltamp_bg
@@ -281,7 +281,7 @@ Item {
         width: scaledBounds.width * sceneItem.width
         height: scaledBounds.height * sceneItem.height
         y: scaledBounds.y * sceneItem.height
-        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled())
+        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled() && UAV.batteryADCConfigured())
 
         Rectangle {
             anchors.fill: parent
@@ -296,7 +296,7 @@ Item {
         width: scaledBounds.width * sceneItem.width
         height: scaledBounds.height * sceneItem.height
         y: Math.floor(scaledBounds.y * sceneItem.height)
-        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled())
+        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled() && UAV.batteryADCConfigured())
     }
 
     SvgElementPositionItem {
@@ -307,7 +307,7 @@ Item {
         width: scaledBounds.width * sceneItem.width
         height: scaledBounds.height * sceneItem.height
         y: scaledBounds.y * sceneItem.height
-        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled())
+        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled() && UAV.batteryADCConfigured())
 
         Rectangle {
             anchors.fill: parent
@@ -334,7 +334,7 @@ Item {
         width: scaledBounds.width * sceneItem.width
         height: scaledBounds.height * sceneItem.height
         y: scaledBounds.y * sceneItem.height
-        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled())
+        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled() && UAV.batteryADCConfigured())
 
         Rectangle {
             anchors.fill: parent
@@ -361,7 +361,7 @@ Item {
         width: scaledBounds.width * sceneItem.width
         height: scaledBounds.height * sceneItem.height
         y: scaledBounds.y * sceneItem.height
-        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled())
+        visible: (!UAV.isPathPlanEnabled() && UAV.batteryModuleEnabled() && UAV.batteryADCConfigured())
 
         Rectangle {
             anchors.fill: parent