Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
olliw42 opened this issue Jul 1, 2023 · 4 comments

Comments

@olliw42
Copy link
Owner

olliw42 commented Jul 1, 2023

follow up on #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.
@olliw42 olliw42 changed the title Lua for BW screens: Support more than 5 parameters Lua for BW screens: Support more than 5 parameters, and other improvements Jul 2, 2023
@geofrancis
Copy link
Contributor

geofrancis commented Jul 18, 2023

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
Copy link
Owner Author

olliw42 commented Jul 18, 2023

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
Copy link

zanefond commented Aug 5, 2023

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

@galaxyrepairman
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants