Lua for BW screens: Support more than 5 parameters, and other improvements

olliw42 / mLRS

2.4 GHz & 915/868 MHz & 433 MHz/70 cm LoRa based telemetry and radio link for remote controlled vehicles

GNU General Public License v3.0

295 stars 67 forks source link

Lua for BW screens: Support more than 5 parameters, and other improvements #89

Open olliw42 opened 1 year ago

olliw42 commented 1 year ago

follow up on https://github.com/olliw42/mLRS/issues/74.

The current lua script for bw screens supports only 5 parameters, for reasons of limited ram on radios with bw screen. It is however possible to read parameters also one by one, by using the parameter-by-index command, which should allow extended functionality on bw screen radios.
The handled parameters are hardcoded by their index. If the setup layout changes the index changes, breaking the functioning.

geofrancis commented 1 year ago

I noticed that when i had the wrong firmware it would display the wrong parameters, that suggests it can display any parameter but only 5 at once. would it be possible for the user to select the index numbers for the parameters they want access to? so it would be possible to select what 5 parameters it displays.

also im struggling to get it to connect reliably, is there any prerequisites for using the lua. ? I generally find I have to restart everything a couple if times for it to work.

olliw42 commented 1 year ago

would it be possible for the user to select the index numbers for the parameters they want access to? so it would be possible to select what 5 parameters it displays.

the 5 indices are defined here: https://github.com/olliw42/mLRS/blob/main/lua/mLRS-bw.lua#L27-L31 so it might be that all you need to do is to change them as you like just a guess though

also im struggling to get it to connect reliably, is there any prerequisites for using the lua. ? I generally find I have to restart everything a couple if times for it to work.

pl note that we do not appear to have any maintainer for that bw lua script. This also implies that it may function worse and worse over time when mLRS progresses. I probably was not a wise decisssion to "officially" include it.

zanefond commented 1 year ago

the new lua script works well on my frsky x9d+ （ edgetx sys and resistance modified ）

galaxyrepairman commented 9 months ago

Hello, may I ask which signal line is used as the output of CRSF for E77 MBLkit TX? Is the E77 MBLkit successfully aligned? LED 3 will light up