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

feat(radio): Allow precision for mix slow up/dn to be set to 10ms or 100ms. #4612

Merged
merged 2 commits into from
Feb 12, 2024

Conversation

philmoz
Copy link
Collaborator

@philmoz philmoz commented Feb 7, 2024

Fixes #4593

Support the range for slow up/down on a mix line to be 0-2.5s in 10ms increments.

@3djc
Copy link
Collaborator

3djc commented Feb 8, 2024

I would like to add some gtest, but can do as a separate PR

@philmoz
Copy link
Collaborator Author

philmoz commented Feb 9, 2024

I've duplicated the existing tests and modified them to use the 10ms precision.

@pfeerick pfeerick added this to the 2.10 milestone Feb 11, 2024
@pfeerick pfeerick added the enhancement ✨ New feature or request label Feb 11, 2024
@pfeerick pfeerick self-requested a review February 11, 2024 23:17
@pfeerick
Copy link
Member

LGTM on X9D+, MT12 and TX16S. I'm still a little unsure if showing it as 0.00 / 0.0 is the most intuitive, but we'll just have to see what happens with usage.

@pfeerick pfeerick merged commit 84c2d85 into EdgeTX:main Feb 12, 2024
46 checks passed
@pfeerick pfeerick mentioned this pull request Mar 11, 2024
13 tasks
@philmoz philmoz deleted the mix-slow-prec branch May 10, 2024 04:47
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increase precision of Slow Up/Slow Down for surface controllers
3 participants