betaflight / betaflight-tx-lua-scripts

Collection of scripts to configure Betaflight from your TX (currently only supported in OpenTx)
GNU General Public License v3.0
590 stars 142 forks source link

Hide OSD (SD) settings #496

Closed atomgomba closed 5 months ago

atomgomba commented 7 months ago

Follow up PR on #495. Please merge #495 first, because it contains the parent commit.

atomgomba commented 7 months ago

It looks like on some boards getting the OSD config may take a lot of time over CRSF

TheIsotopes commented 7 months ago

Tested and everything seems to work.

Imo a rebase is still missing here.

atomgomba commented 7 months ago

Tested and everything seems to work.

Imo a rebase is still missing here.

Seemingly it doesn't always work for me. I tested using 4 different quads. My radio is Tango 2 with FreedomTX, what's your radio and firmware? I'm starting to feel like I'm gonna need more hardware for testing. I can get a Micro TX for my Taranis X9D+ (has EdgeTX on it) to have one more test device.

TheIsotopes commented 7 months ago

I use the RM Zorro with EdgeTX for FrSky and Ghost receivers.

atomgomba commented 7 months ago

With #13333 we're slowly getting to the point where firing a single MSP request to get the supported options will be enough. Waiting for multiple responses takes inconveniently lot amount of time, I'm planning to put this PR on hold until we have a better way to check build options.

klutvott123 commented 7 months ago

With #13333 we're slowly getting to the point where firing a single MSP request to get the supported options will be enough. Waiting for multiple responses takes inconveniently lot amount of time, I'm planning to put this PR on hold until we have a better way to check build options.

This is probably the best solution. I think we should put everything else on hold just to avoid ending up with multiple solutions and compatibility issues. We have three solutions to this right now, and we should pick just one. I'm not sure which protocol you're testing this with, but the worst case scenario is FPORT as it's super slow.

atomgomba commented 7 months ago

With #13333 we're slowly getting to the point where firing a single MSP request to get the supported options will be enough. Waiting for multiple responses takes inconveniently lot amount of time, I'm planning to put this PR on hold until we have a better way to check build options.

This is probably the best solution. I think we should put everything else on hold just to avoid ending up with multiple solutions and compatibility issues. We have three solutions to this right now, and we should pick just one. I'm not sure which protocol you're testing this with, but the worst case scenario is FPORT as it's super slow.

Currently I've only tested on CRSF, but on two "different" platforms, FreedomTX and EdgeTX. I happen to own an X9D+ and FrSky receivers so with a bit of soldering I can test FPORT too. Thanks for the heads up!

atomgomba commented 7 months ago

New plan is to have a dedicated MSP command for clients using OTA transmission, please see #13344

EDIT: Above PR #13344 had been dropped, will go ahead with #13333 instead

atomgomba commented 6 months ago

This is not stale, just waiting for #13333 to be merged. I presume it's most likely to happen after 4.5 release.

haslinghuis commented 5 months ago

@TheIsotopes pls test again :) (needs latest dev firmware)

atomgomba commented 5 months ago

@TheIsotopes pls test again :) (needs latest dev firmware)

well, it's only needed to test the happy path. by design the menu will load either way ;) btw I also tested this, but more tests the better. testing against MSP version <= 1.46 is a real scenario as well. so far I haven't tested against BF 4.4, but I'm planning to do so in the weekend

TheIsotopes commented 5 months ago

@atomgomba @haslinghuis tested on my Zorro, everything seems to be working correctly for me. 👍

atomgomba commented 5 months ago

tested on 4.4.3, works as expected