ESC Telemetry Dashboard Widget#1040
Merged
1Blademaster merged 26 commits intomainfrom Mar 25, 2026
Merged
Conversation
…t scales for > 4 esc's
Contributor
There was a problem hiding this comment.
Pull request overview
Adds an ESC telemetry dashboard widget and wires MAVLink ESC telemetry messages through the backend state listeners and the GCS Redux/socket pipeline so the dashboard can display ESC metrics alongside existing widgets.
Changes:
- Subscribe the backend dashboard state to
ESC_TELEMETRY_1_TO_4and update the corresponding test expectation. - Extend GCS Redux state/selectors and socket middleware to store/aggregate ESC telemetry packets.
- Add a new
EscTelemetryWidgetand adjust the dashboard layout to stack widgets at the bottom.
Reviewed changes
Copilot reviewed 8 out of 10 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| yarn.lock | Adds a Yarn v1 lockfile header at repo root. |
| node_modules/.yarn-integrity | Adds Yarn integrity metadata file under root node_modules/. |
| radio/tests/test_states.py | Updates expected listener count for dashboard state. |
| radio/app/endpoints/states.py | Adds ESC_TELEMETRY_1_TO_4 to dashboard message listeners. |
| radio/app/endpoints/params.py | Formats params_error emit payload into a one-liner. |
| gcs/src/redux/slices/droneInfoSlice.js | Adds ESC telemetry state, reducers/selectors, and an aggregated selector. |
| gcs/src/redux/middleware/socketMiddleware.js | Dispatches ESC telemetry messages into Redux on receipt. |
| gcs/src/dashboard.jsx | Adds the ESC telemetry widget and groups bottom widgets in a stacked container. |
| gcs/src/components/dashboard/videoWidget.jsx | Refactors positioning; widget now relies on parent container layout. |
| gcs/src/components/dashboard/EscTelemetryWidget.jsx | Introduces the new ESC telemetry widget UI with threshold settings and resizing. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1Blademaster
requested changes
Mar 12, 2026
Member
https://github.com/Avis-Drone-Labs/FGCS into 979-feature-have-a-dashboard-widget-for-esc-telemetry
Contributor
Author
Member
Member
1Blademaster
approved these changes
Mar 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.




No description provided.