Hardware OmniNXT F7, Crossfire Nano/Micro combo, telemetry over CRSF

I think it's this in the lua scripts:

-- editing value
elseif currentState == pageStatus.editing then
    if event == EVT_VIRTUAL_EXIT or event == EVT_VIRTUAL_ENTER then
        currentState = pageStatus.display
    elseif event == EVT_VIRTUAL_INC or event == EVT_VIRTUAL_INC_REPT then
        incValue(1)
    elseif event == EVT_VIRTUAL_DEC or event == EVT_VIRTUAL_DEC_REPT then
        incValue(-1)
        killEvents(event)
    end
end

@raphaelcoeffic any idea why there is a killEvents here?

@frozenskys darn, which file is it in? I searched for EVT_VIRTUAL_DEC_RPT but could only find a symmetrical code snippet.

I think there has been problems of ''double key press' events being generated uncommanded. Pbly this is a leftover of this after it has been fixed by other means / in OTX meanwhile?

Could be left from before the change to virtual events - this is in the SCRIPTS\BF\ui.lua file - I'll need to check the history when I get back from work - I'm limited to what I can do with code right now in the office... Also I checked latest master so not sure if it's in 1.4.x

Yeah, looks like a left over to me as well. Or maybe on some platform it needs to not generate another key press... hard to remember what I did. Could you please try to check with “git blame” where this is coming from?

@raphaelcoeffic commit 86860af

I deleted the killEvents line, but issue remains. Didn't find another occurence of EVT_VIRTUAL_DEC_REPT in the ui.lua file...

Unfortunately I only have an X7 which doesn't generate the EVT_VIRTUAL_DEC_REPT event 😢 and I haven't worked out how to get the telemetry working with the OTX simulator to test this on a simulated X9

Tried again by installing 1.4.1 release, then deleting this line from it --> worked!

Seems I had pulled from master before, and for some reason this didn't.

@raphaelcoeffic so this line needs to go, and there are no side effects to be expected?

Looks like this was a fix for an issue pre virtual events - see #245 and #251
so I'd assume it's no longer needed.

PR created.

@RipperDrone @frozenskys Does it work properly on X9D+ without this line?

For me it does - yep. (OTX 2.3.1, Lua 1.4.1 without this line, Taranis X9D+ 2013, BF4.1 Release)