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

ppmus build does not display ppmus in Outputs #3163

Closed
1 task done
sande005 opened this issue Feb 9, 2023 · 1 comment · Fixed by #3993
Closed
1 task done

ppmus build does not display ppmus in Outputs #3163

sande005 opened this issue Feb 9, 2023 · 1 comment · Fixed by #3993
Labels
bug 🪲 Something isn't working

Comments

@sande005
Copy link

sande005 commented Feb 9, 2023

Is there an existing issue for this problem?

  • I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

Buried in the custom build thread (2362) are comments for implementing it on a Frsky X10SE, Edge ver 2.8.0
Build completes normally. After install, Min/Max on the Outputs page still display in terms of per cent.

Suggest for future enhancement that since this is primarily just a numeric conversion for display purposes, that it become an option that can be toggled on/off at will in the radio, rather than depending on a new firmware build.

Expected Behavior

Outputs display settings with ppmus values rather than percents

Steps To Reproduce

Build custom firmware with the -ppmus option, and install.

Version

2.8.0

Transmitter

FrSky X10 Express / X10S Express (ACCESS)

Anything else?

No response

@sande005 sande005 added bug 🪲 Something isn't working triage Bug report awaiting review / sorting labels Feb 9, 2023
@raphaelcoeffic
Copy link
Member

Totally agreed, we should have this as a settings, rather than a compilation argument.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants