-
-
Notifications
You must be signed in to change notification settings - Fork 154
Remove TELEMETRY script #306
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
Remove TELEMETRY script #306
Conversation
Removes the telemetry version of the script. bf.lua still exists but only displays a message telling the user to use the TOOLS version instead. bf.lua should be removed from the TELEMETRY folder at a later time.
One question that I have: Should the scripts in I noticed that synchronisation of the RTC stopped working with the removal of the telemetry script. |
@mikeller It has to be set up as a special or global function. I just set it to run all the time |
@klutvott123: Ah I see, and now it's working. In this case it probably makes sense to separate the RTC function into a separate script, so that users who want RTC functionality but do not need RSSI to be injected periodically (taking up telemetry bandwidth) can do so. |
So nobody is ever flying while using the BF tools script? If he uses script on other places, they stop working during tool operation. Just be informed |
@mikeller I agree. I suspect that most people only need the RTC functionality anyway. I'll see if I can split it up. @lshems I hope not. I support using scripts like this while flying just as much as I support texting while driving. I mentioned in the first post that all scripts are stopped when running one-time scripts. |
@klutvott123 I'm using the telemetry script while flying a lot, to display life tele values on my Taranis RC and to trigger haptic vibration alarms for battery low state so I can feel it when flying LOS or older quads without OSD and with no/weak buzzers onboard. Will this still be possible with a one-time script? If not it means sacrificing some well working functionality for me... :-/ |
@klutvott123: Not sure what you mean by 'trigger haptic vibration alarms for battery low state' - this was never something that the Betaflight TX lua scripts did. Are you confusing this with the normal reception of telemetry information, and the capability to set alarms based on them that is part of OpenTX? |
@mikeller oh yeah, that might well be right :-). I'm doing both: Having all telemetry sensor information displayed on the Taranis screen inflight (which is by telemetry script), plus I have coded some alarms based on sensor information received. So by what you clarified for me now, the more accurate question is: How can the telemetry sensor information still be displayed on the Taranis screen inflight when it is changed from telemetry script mechanism to one-time telemetry script instead? |
This is also not something that the Betaflight TX lua scripts ever did - please ask these questions whoever is maintaining the script that you are using. |
It's just the Betaflight telemetry script that has been removed. All other telemetry/function scripts you might have should keep working as they always have. Logical switches and special functions will continue to work even if a one time script is running. |
@mikeller seems i was wrong in my understanding of the concept. There is no other scripts I could ask anybody about. However, I was mistaken by the bf.lua script set as a telemetry page - thinking this is taking care of telemetry sensor values being displayed (which it is not. It's a core OTX functionality rather, it seems). All clear now by explanations given by @klutvott123. Thanks again! |
As discussed in #302
Removes the telemetry version of the script.
bf.lua still exists but only displays a message telling the user to use the TOOLS version instead. bf.lua should be removed from the TELEMETRY folder at a later time.
Now that we have the script available as a one-time script in the tools menu, there aren't many good reasons to keep the telemetry version IMO. There are many benefits to just having the tools version.
Uses less memory.
Frees up a lot of memory that can be used for actual telemetry scripts.
All scripts are stopped when starting one-time script. This means more available memory for BF lua.
Keys work as expected.
Avoids potential conflicts with other scripts using the opentx telemetryPush/Pop functions.