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

Can't Change PID Values ? #305

Closed
aburn09 opened this issue Jan 25, 2020 · 11 comments
Closed

Can't Change PID Values ? #305

aburn09 opened this issue Jan 25, 2020 · 11 comments

Comments

@aburn09
Copy link

aburn09 commented Jan 25, 2020

Hi Just updated my Taranis radio and bootloader X9D+ SE on Open Tx to the latest 2.3.5 version also updated the SD card and Lua scripts to 1.4.1 but now I can't change my PID values on the tx everytime I press page on the tx it shows the rssi value pressing again gives me the lua script PID settings but when I press ENT on my TX then either the plus or minus button on the TX the page returns to the rssi page again ?
Before I updated everything was working correctly but now reverting back to the backed up version still has the same results !!
This is a short youtube clip showing the problem ?

https://www.youtube.com/watch?v=mZf6NbP98JA

Quick update it seems on page 13 of the tx I have two telemetry screens enabled.....Screen 1 is for Scripts and Screen 2 was for RSSI and battery voltage......If I disable screen 2 then Lua scripts work ? Any thoughts apart from only having Screen 1 enabled ?

@klutvott123
Copy link
Member

I believe this is an issue with opentx 2.3.5. +/- now does the same as the PAGE key for some reason. It has been reported here opentx/opentx#7309 too.

@aburn09
Copy link
Author

aburn09 commented Jan 25, 2020

Thanks klutvott123 I suppose I'll have to only have one telemetey page ie just my pids until they fix the problem ?

@klutvott123
Copy link
Member

Yes. Another option is to try the latest betaflight lua master and launch the script from the tools menu. All keys work as they should in that version. If you decide to try it, I recommend removing all files from earlier versions first as there has been a lot of changes since 1.4.1

@RipperDrone
Copy link
Contributor

Confirming issue, OTX2.3.5 messes up +/- key functions in lua script

@mikeller
Copy link
Member

@RipperDrone:

Confirming issue, OTX2.3.5 messes up +/- key functions in lua script

Is this when running the script as a telemetry script?

@aburn09
Copy link
Author

aburn09 commented Jan 26, 2020

@RipperDrone yes if you run more than one Telemetry script page there seems to be a conflict with the plus and minus button on the taranis x9d+ ?

@mikeller
Copy link
Member

@aburn09: Running the Betaflight configuration script as a telemetry script is deprecated as it has a number of issues (key bindings not being supported in this mode among them), and support for it will be dropped in the next release (#306).

@aburn09
Copy link
Author

aburn09 commented Jan 26, 2020

mikeller can I assume then that if I take my tx back one or two versions of open tx I can get two or more telemetry script pages working again as before I updated to 2.3.5 ?

@mikeller
Copy link
Member

@aburn09: You should ask this in https://github.com/opentx/opentx/issues/ - as stated in #305 (comment) Betaflight does not support telemetry scripts any more.

@RipperDrone
Copy link
Contributor

RipperDrone commented Jan 26, 2020

@mikeller yes, it's with a telemetry script. Using it to invoke the Taranis' gimbal as a low voltage alarm, and to tune PIDs etc.

If scripts are going to be dropped in favor of several /tools/* one-time scripts, can those scripts still be linked to a telemetry page and execute by long pressing the page button on the Taranis? Or do we have to invoke the script manually and individuallyl anytime we need to access a specific function? Not quite clear to me...

@mikeller
Copy link
Member

@RipperDrone: See #305 (comment).

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

No branches or pull requests

4 participants