LuaTelemetry icon indicating copy to clipboard operation
LuaTelemetry copied to clipboard

Support OpenTX's new virtual key events

Open teckel12 opened this issue 4 years ago • 5 comments

Is your feature request related to a problem? Please describe

OpenTX v2.3 adds new virtual key events that simplifies the controls of multiple controllers.

Describe the solution you'd like

Clean up Lua Telemetry code to use the virtual key events

Describe alternatives you've considered

Continue the painful way it's being done now, where every controller needs to be hand-built.

Additional context

For Lua Telemetry to continue to work with older versions of OpenTX, detecting of missing virtual key events will be required. Another option is to require OpenTX 2.3, but I don't think that can be done till a later date (but will be done at some point so legacy code can be removed). Maybe 6 months after OpenTX 2.3 is released it's made a requirement of Lua Telemetry?

teckel12 avatar Sep 23 '19 14:09 teckel12

I'll wait till the beginning of 2020 to switch over to using the virtual key events. Seems EVT_VIRTUAL_MENU is missing from the Jumper T12 and there could be other issues I'm not aware of at this time. By January of 2020 it's not unrealistic to require OpenTX 2.3+

teckel12 avatar Sep 30 '19 17:09 teckel12

Jumper T12, INav and OpenTX 2.3.4, new key config, naming, mapping, configuration of the T12s 6 buttons has a different instruction set in OpenTX. I imagine this is why only the enter key works with INav now. Sounds like a simple fix. Happy Holidays

LawrenceLDJr avatar Dec 31 '19 00:12 LawrenceLDJr

Is there a branch anywhere that contains the required changes for OpenTx 2.3.5 compatibility ?

I don't know anything about LUA or how to write it, but if its a find/replace I might be able to add the changes, maybe... My USB port on the flight controller is not really accessible in my model so would really like to use this as soon as its available.

jj-uk avatar Feb 19 '20 14:02 jj-uk

@jj-uk There's not currently a branch with support for the new virtual key events. It's more than just a simple search and replace. The change will break anyone using OpenTX prior to v2.3 which is why it hasn't been changed yet. Once the switch has happened, OpenTX 2.3 will be required. This spring (as it's currently too cold to fly where I'm from).

teckel12 avatar Feb 25 '20 20:02 teckel12

There's a fix in Open-tx 2.3.6 that addresses issues with the + - Exit buttons. Not sure if this is related? https://github.com/opentx/opentx/pull/7356

jj-uk avatar Apr 01 '20 15:04 jj-uk