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

Hide OSD (SD) settings #496

Merged
merged 1 commit into from
Mar 23, 2024

Conversation

atomgomba
Copy link
Contributor

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

@atomgomba
Copy link
Contributor Author

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

@TheIsotopes
Copy link

Tested and everything seems to work.

Imo a rebase is still missing here.

@atomgomba
Copy link
Contributor Author

atomgomba commented Jan 25, 2024

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.

@atomgomba atomgomba force-pushed the hide-osd-sd-settings branch from 9b96921 to 7a845bc Compare January 25, 2024 10:19
@TheIsotopes
Copy link

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

@atomgomba
Copy link
Contributor Author

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.

@atomgomba atomgomba force-pushed the hide-osd-sd-settings branch 5 times, most recently from 667132c to 5b8891e Compare February 2, 2024 17:13
@klutvott123
Copy link
Member

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

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

atomgomba commented Feb 4, 2024

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 atomgomba force-pushed the hide-osd-sd-settings branch from a97034e to 2acf21e Compare March 3, 2024 09:59
@atomgomba
Copy link
Contributor Author

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

@atomgomba atomgomba force-pushed the hide-osd-sd-settings branch from 2acf21e to 4a7ec40 Compare March 17, 2024 19:05
@haslinghuis haslinghuis added this to the 1.8 milestone Mar 17, 2024
@atomgomba atomgomba force-pushed the hide-osd-sd-settings branch from 4a7ec40 to 61e8f4c Compare March 22, 2024 14:13
@atomgomba atomgomba marked this pull request as ready for review March 22, 2024 14:13
@haslinghuis
Copy link
Member

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

@atomgomba
Copy link
Contributor Author

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

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

@atomgomba
Copy link
Contributor Author

tested on 4.4.3, works as expected

@haslinghuis haslinghuis merged commit de29b66 into betaflight:master Mar 23, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants