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

Fix pots and sliders warning #1277 #1281

Merged
merged 1 commit into from
Dec 28, 2021
Merged

Conversation

eshifri
Copy link
Contributor

@eshifri eshifri commented Dec 21, 2021

Fixes #1277

Summary of changes: it was rather messy. :-)

@pfeerick
Copy link
Member

"Here be dragons"? 🐉 ⚔️ 😆

@eshifri
Copy link
Contributor Author

eshifri commented Dec 21, 2021

Well, it is difficult to say what was not wrong there. :-)
The assumptions about what is saved in the model were different in different files. Model_setup assumed that non-existing pots are skipped; switch_warning_dialog and switches - that they are just ignored.
Data type uint8_t was insufficient to keep all 11 bits for 11 pots.
And some other incorrect assumptions in the modl_setup.
I'm afraid looking at the code changes is the shortest way to find it all out. :-)

@pfeerick
Copy link
Member

I'm not worried about that... just when I see a comment like "it was rather messy" as the summary I know there are good times ahead! Top marks... We need a contest for the most... entertaining... PR summary! Need some levity after delving in the dark bowels of the code. 😆

@raphaelcoeffic raphaelcoeffic added this to the 2.6 milestone Dec 23, 2021
@pfeerick pfeerick self-assigned this Dec 27, 2021
@pfeerick pfeerick added bug 🪲 Something isn't working color Related generally to color LCD radios labels Dec 27, 2021
Copy link
Member

@pfeerick pfeerick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you... I get it now after reading the code 😁 That explains the odd behaviour I saw with the RS warning being shown, but never actually going away, even though the condition was met.

@pfeerick pfeerick merged commit 87724bb into EdgeTX:main Dec 28, 2021
pfeerick pushed a commit that referenced this pull request Dec 28, 2021
Some pot/sliders were being ignored or check state was inverted
stephenaa pushed a commit to stephenaa/edgetx that referenced this pull request Dec 28, 2021
Some pot/sliders were being ignored or check state was inverted
@eshifri eshifri deleted the eshifri/pot_warning branch December 29, 2021 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something isn't working color Related generally to color LCD radios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pots and Sliders check buttons are reversed.
3 participants