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.

Sign up for GitHub

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

Jump to bottom

fix(radio): Don't allow negative values for V2 on LS functions '|x|>y', '|x|<y' and '|dx|>=y' #4239

Merged
merged 1 commit into from
Nov 12, 2023
Merged

fix(radio): Don't allow negative values for V2 on LS functions '|x|>y', '|x|<y' and '|dx|>=y' #4239

merged 1 commit into from
Nov 12, 2023

Conversation

philmoz
Copy link
Collaborator

@philmoz philmoz commented Oct 25, 2023

Fixes #4238

Limit V2 range to positive values, where function compares to |V1|.

@pfeerick pfeerick added bug 🪲 Something isn't working UX-UI Related to user experience (UX) or user interface (UI) behaviour labels Nov 11, 2023
@pfeerick pfeerick added this to the 2.10 milestone Nov 11, 2023
@pfeerick pfeerick force-pushed the fix-ls-abs-func-range branch from 5b5004f to e2e1de4 Compare November 12, 2023 22:47
@pfeerick pfeerick self-requested a review November 12, 2023 22:47
@pfeerick pfeerick merged commit c811145 into EdgeTX:main Nov 12, 2023
37 checks passed
@philmoz philmoz deleted the fix-ls-abs-func-range branch February 17, 2024 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pfeerick pfeerick Awaiting requested review from pfeerick

Assignees
No one assigned
Labels
bug 🪲 Something isn't working UX-UI Related to user experience (UX) or user interface (UI) behaviour
Projects
None yet
Milestone
2.10
Development

Successfully merging this pull request may close these issues.

Logical switch absolute change allows negative values
2 participants
@philmoz @pfeerick