edgetx icon indicating copy to clipboard operation
edgetx copied to clipboard

Not so much a bug as a switch test missed

Open MervDale opened this issue 3 years ago • 4 comments

Is there an existing issue for this problem?

  • [X] I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

In the switch test sequence , Radio Settings, Hardware, Switches.... all switches are tested but the P 1 to 6 Butttons are not,

Expected Behavior

Should be included

Steps To Reproduce

Radio Settings, Hardware, Switches

Version

Other (Please specify below)

Transmitter

Radiomaster TX16S

Anything else?

Nightlies I'm not sure if this has been mentioned here but I remember hearing from another user.

MervDale avatar Aug 03 '22 11:08 MervDale

I'm not sure if this will happen since the 1-6 buttons are actually an analog input like the pots and sliders which is treated like a set of buttons. So is tested on the analogs page (06 for the TX16S). It would only make sense to do so if we are able to make it so you don't need to also calibrate it along with the other analogs.

pfeerick avatar Aug 03 '22 12:08 pfeerick

Yes I guess you are right there, they aren't quite so obvious in the analogs showing up as only one entry but yes you do need to calibrate them and they are not really switches which you don't have to calibrate.

MervDale avatar Aug 03 '22 12:08 MervDale

It should be possible to use them without calibration. There can't be much different in the voltage the 6-POS MCU generates between the different radios.

gagarinlg avatar Sep 15 '22 11:09 gagarinlg

It should be possible to use them without calibration. There can't be much different in the voltage the 6-POS MCU generates between the different radios.

This is something we should really consider. Calibrating these 6POS is really a pain.

raphaelcoeffic avatar Sep 15 '22 11:09 raphaelcoeffic

Not to mention that it is impossible to 'Center' an even number of switches. Since they are discrete analog values they should not be allowed where a true or false is expected. For instance we can use 6P-1, 6P-2.. to select Flight Modes.
But changing the chicken feet does not change Flight Modes. Also presently I can't find how to put any 6P, into a widget so it's value can be displayed.

Either treat it like an analog or as 6 true or false values, please!.

Well,I was going to attach .etx to show this, but .etx is not allowed attachment type.

Tested in 2.8.4 EdgeTxCompanion only. ThrSafe.pdf ThrSafe-RM-TX16S-2023July16-Rdy03.zip

rdeanchurch avatar Jul 17 '23 00:07 rdeanchurch