-
-
Notifications
You must be signed in to change notification settings - Fork 146
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
Add (e)RPM black box field #640
Conversation
- Add RPM to enabled/disabled black box fields in header dialog - Add escapeRegExp function to tools.js - Escape nameRoot before using as regexp for creating [all] fields in graph_config
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Do you want to test this code? Here you have an automated build: |
AUTOMERGE: (FAIL)
|
@tbolin , Can you update description with details and screencap for prosperity sake? TY |
I've added a bit of detail and some screenshots |
Adds formatting of the RPM field from betaflight/betaflight#12823
The field should show up as just RPM and be decoded as RPM and Hz.
The RPM plots are scaled so that they have the correct height relative to each other, so the same height corresponds to the same RPM for all plots.
Also adds an option in Configure graphs to add a graph with all RPM plots.
Depends on https://github.com/betaflight/betaflight/pull/12823