Add reseting PWM field values on button select combo change

Author: zdanek

src/Joystick/Joystick.cc

@@ -123,13 +123,13 @@ void AssignedButtonAction::sendPwm(Vehicle *vehicle, bool buttonDown)
         if (buttonDown) {
             _pwmLatchButtonDown = !_pwmLatchButtonDown;
             pwmValue = _pwmLatchButtonDown ? _hiPwmValue : _loPwmValue;
-            qCDebug(JoystickLog) << " calculated new PWM Value " << pwmValue;
         } else {
-            qCDebug(JoystickLog) << "since button up - exiting";
+            qCDebug(JoystickLog) << "since button is up - exiting";
             return;
         }
     }
 
+    qCDebug(JoystickLog) << " Sending PWM Value " << pwmValue;
     vehicle->rcChannelOverride(_pwmRcChannel, pwmValue);
 }
 

src/Vehicle/Vehicle.cc

@@ -2219,7 +2219,8 @@ void Vehicle::requestDataStream(MAV_DATA_STREAM stream, uint16_t rate, bool send
 void Vehicle::_sendMessageMultipleNext()
 {
     if (_nextSendMessageMultipleIndex < _sendMessageMultipleList.count()) {
-        qCDebug(VehicleLog) << "_sendMessageMultipleNext:" << _sendMessageMultipleList[_nextSendMessageMultipleIndex].message.msgid;
+//bzd to be enabled but it spams too much
+         //qCDebug(VehicleLog) << "_sendMessageMultipleNext: " << _nextSendMessageMultipleIndex << ", mesg id: " << _sendMessageMultipleList[_nextSendMessageMultipleIndex].message.msgid;
 
         SharedLinkInterfacePtr sharedLink = vehicleLinkManager()->primaryLink().lock();
         if (sharedLink) {
@@ -4213,7 +4214,9 @@ void Vehicle::rcChannelOverride(uint8_t rcChannel, uint16_t pwmValue)
         override_data[i] = UINT16_MAX;
     }
     override_data[rcChannel - 1] = pwmValue;
-
+    for (int i = 0; i < 18; i++) {
+        qCDebug(VehicleLog) << "Override data " << i << " is " << override_data[i];
+    }
     mavlink_message_t message;
 
     mavlink_msg_rc_channels_override_pack_chan(

src/VehicleSetup/JoystickConfigButtons.qml

@@ -144,11 +144,26 @@ ColumnLayout {
                             anchors.verticalCenter:     parent.verticalCenter
                         }
                         QGCTextField {
+                            id:     lowPwmValue
                             width:  ScreenTools.defaultFontPixelWidth * 10
                             implicitHeight:     ScreenTools.implicitTextFieldHeight
                             visible:                true
                             validator: IntValidator { bottom:1000; top: 2000}
 
+                            Connections {
+                                target: buttonActionCombo
+                                onCurrentIndexChanged: {
+                                    if(_activeJoystick) {
+                                        console.log("index changed, ", buttonActionCombo.currentIndex)
+                                        console.log("index changed, ", modelData)
+                                        console.log("index changed, ", target)
+                                        var pwm = pwmSettings._getButtonPwm(modelData, true)
+                                        console.log("pwm ", pwm)
+                                        lowPwmValue.text = pwm;
+                                    }
+                                }
+                            }
+
                             Component.onCompleted: {
                                 if(_activeJoystick) {
                                     text = parent._getButtonPwm(modelData, true)
@@ -163,11 +178,27 @@ ColumnLayout {
                             anchors.verticalCenter:     parent.verticalCenter
                         }
                         QGCTextField {
+                            id:     highPwmValue
                             width:  ScreenTools.defaultFontPixelWidth * 10
                             implicitHeight:     ScreenTools.implicitTextFieldHeight
                             visible:                true
                             validator: IntValidator { bottom:1000; top: 2000}
 
+                            Connections {
+                                target: buttonActionCombo
+                                function onCurrentIndexChanged(index) {
+                                    if(_activeJoystick) {
+                                        console.log("index changed, ", buttonActionCombo.currentIndex)
+                                        console.log("index changed, ", modelData)
+                                        console.log("index changed, ", target)
+                                        console.log("text ", target.text)
+                                        var pwm = pwmSettings._getButtonPwm(modelData, false)
+                                        console.log("pwm ", pwm)
+                                        highPwmValue.text = pwm;
+                                    }
+                                }
+                            }
+
                             Component.onCompleted: {
                                 if(_activeJoystick) {
                                     text = parent._getButtonPwm(modelData, false)