fix(lua): stick "side" missing from model.getInput

[forbesmyester]

Model > Input -> Side is missing from the Lua API.

This PR puts it back in.

NOTE: I think this is correct, I've been able to compile firmware, but currently I am unable to get the simulator to compile on a Ubuntu22 VM. It dies with the following error and my C++ skills are not going to be sufficient to move forward.

In file included from /home/user/edgetx/edgetx_main/companion/src/firmwares/eeprominterface.h:27,
                 from /home/user/edgetx/edgetx_main/companion/src/storage/eepe.cpp:24:
/home/user/edgetx/edgetx_main/build-output/native/../../radio/src/definitions.h:28:10: fatal error: memory_sections.h: No such file or directory
   28 | #include "memory_sections.h"
      |          ^~~~~~~~~~~~~~~~~~~
compilation terminated.

[forbesmyester]

If I wasn't clear, the actual firmware does compile, but that's all the testing I could do, I spent the whole morning trying with companion.

[pfeerick]

No, that was clear... I've just rebased this on the current state of main as it was from before a fix was pushed for the just-broken Companion builds. So Companion should work for you again locally if you reset your local branch to the remote branch.

[pfeerick]

Ok, so this is working fine on GX12, is returning/displaying 3, 2 or 1 depending on the side mode set. Should this be mode or perhaps instead side?

-- "TNS|PR5801 Test|TNE"

local modelInput
local exit = 0

local function init()
  modelInput = model.getInput(0,0)
end

local function run(event, touchState)
  lcd.clear()

  lcd.drawText( 0, 0, "PR5801 modelInput.mode")
  lcd.drawText( 0, 20, "Mode : " .. tostring(modelInput.side) )

  if event == EVT_VIRTUAL_EXIT then 
    exit = 1
  end 
  return exit
end

return { run=run, init=init }

[forbesmyester]

Perfectly happy with either. I did think about that but instead documented that it was side.

Happy to make the change if you want

[pfeerick]

Yeah, I think it would be a good idea to call it `side` or even something like `stickSide` so it's self-explanatory what it relates to. Dealers choice lol

…

On Tue, Jan 21, 2025 at 4:55 PM Matt Forrester ***@***.***> wrote: Perfectly happy with either. I did think about that but instead documented that it was side. Happy to make the change if you want — Reply to this email directly, view it on GitHub <#5801 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABJ66KNA7S24G5XY57WNEP32LXVODAVCNFSM6AAAAABVNWT7TGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMBTHAYDAOJYGQ> . You are receiving this because you commented.Message ID: ***@***.***>