Add generic WayBionic RViz2 diagnostics foundation package#2
Open
khuzaymahbinharis-jpg wants to merge 19 commits into
Open
Add generic WayBionic RViz2 diagnostics foundation package#2khuzaymahbinharis-jpg wants to merge 19 commits into
khuzaymahbinharis-jpg wants to merge 19 commits into
Conversation
Define DiagnosticMessage and shared status helpers so the UI can consume mock or future ROS 2 data through one interface. Co-authored-by: Cursor <cursoragent@cursor.com>
Provide locally generated DiagnosticMessage data so the dashboard can run without ROS 2 or hardware. Co-authored-by: Cursor <cursoragent@cursor.com>
Centralize Qt stylesheet constants for the monitoring prototype panels, tables, and OK/WARN/FAULT/STALE states. Co-authored-by: Cursor <cursoragent@cursor.com>
Implement the two-column layout, telemetry table, alerts panel, demo mode controls, and reserved camera/visualization placeholders. Co-authored-by: Cursor <cursoragent@cursor.com>
Wire up the runnable PySide6 launcher and document install/run steps plus the backend diagnostics contract for future ROS 2 integration. Co-authored-by: Cursor <cursoragent@cursor.com>
Clarify that the prototype is built with Python 3, PySide6, and Qt Widgets, and note the planned ROS 2 backend integration path. Co-authored-by: Cursor <cursoragent@cursor.com>
Create the isolated ament_cmake package and RViz plugin export metadata for the ground station UI work. Co-authored-by: Cursor <cursoragent@cursor.com>
Define DiagnosticMessage and local Normal/Fault demo diagnostics so the RViz panel can run before live ROS topics are available. Co-authored-by: Cursor <cursoragent@cursor.com>
Implement the docked Qt panel with system status, telemetry table, alerts, and Normal/Fault mock demo controls. Co-authored-by: Cursor <cursoragent@cursor.com>
Provide camera-only and monitoring-first RViz configs plus passive engineer visualization launch without motor commands. Co-authored-by: Cursor <cursoragent@cursor.com>
Add workspace build/run instructions for ar4_ws and describe future diagnostics, camera, and robot visualization integration paths. Co-authored-by: Cursor <cursoragent@cursor.com>
Rename ground_station_monitoring_ui to ground_station_monitoring_ui_archived and note that waybionic_rviz_plugins is now the active ground station UI. Co-authored-by: Cursor <cursoragent@cursor.com>
Move passive AR4 visualization into an optional engineer_ar4_demo launch/config so the default engineer view stays generic. Co-authored-by: Cursor <cursoragent@cursor.com>
Introduce DiagnosticsSource with MockDiagnosticsSource and RosDiagnosticsSource, use_mock_diagnostics launch/config support, and live /diagnostics mapping into DiagnosticMessage. Co-authored-by: Cursor <cursoragent@cursor.com>
Register the second RViz panel, dock it in doctor_camera_view.rviz, and show placeholder topics with a waiting-for-feed status. Co-authored-by: Cursor <cursoragent@cursor.com>
Update package README and docs for the decoupled AR4 helper, SurgeonCameraPanel, and live /diagnostics integration path. Co-authored-by: Cursor <cursoragent@cursor.com>
Store mock normal/fault and surgeon placeholder captures under docs/screenshots and reference them at the top of PR_NOTES. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
yassinsolim
requested changes
Jun 20, 2026
yassinsolim
left a comment
Member
There was a problem hiding this comment.
This needs changes before merge.
The core RViz plugin package builds, and the RViz launch files resolve/start without plugin load errors in local smoke testing. The issue is scope and cleanup.
Change requests:
- Remove the surgeon/doctor camera placeholder work. The surgeon feed should probably be a separate low-latency app or VR workflow, not an RViz placeholder panel.
- Remove
SurgeonCameraPanel, the plugin registration for it, the doctor/camera RViz config, and related docs/screenshots. - Replace the placeholder maintainer info in
waybionic_rviz_plugins/package.xml. - Add real lint/test wiring. Right now
colcon test --packages-select waybionic_rviz_pluginsruns zero tests.
Mac note: RViz crashed on shutdown in RoboStack, but the plugin loaded and the views came up. I would not treat that shutdown crash as a PR blocker unless it also happens on Ubuntu/WSL2.
Remove SurgeonCameraPanel, doctor launch/config, and related docs or screenshots. Add colcon test and lint wiring, replace placeholder maintainer metadata, and add a temporary /diagnostics publisher for mock/live validation while backend publishing is unavailable. Co-authored-by: Cursor <cursoragent@cursor.com>
Member
|
Please fix these before we merge:
|
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.
Review Screenshots
These images are included only to help reviewers see the UI quickly. They are not part of the runtime package behavior.
Summary
This PR makes
waybionic_rviz_pluginsa focused WayBionic RViz2 diagnostics package. It keeps the engineer monitoring panel, mock/live diagnostics switching, and a temporary/diagnosticspublisher for local validation.Camera/doctor placeholder work was removed from this PR and will be handled separately later.
Primary Launch Commands
colcon build --packages-select waybionic_rviz_plugins --symlink-install source install/setup.bash ros2 launch waybionic_rviz_plugins engineer_view.launch.pyMock diagnostics (default):
Live diagnostics:
Temporary backend/demo publisher:
What Works
WayBionic Diagnosticspanel.DiagnosticMessagerows.diagnostic_msgs/msg/DiagnosticArraytopic./diagnosticspublisher withnormal,fault,stale, andcyclemodes.colcon test.DiagnosticsPanelis registered inplugin_description.xml.Testing Performed
Results: build passed; 11 tests, 0 failures; launch argument checks passed; temporary publisher smoke test passed. GUI RViz validation should be re-run locally before final merge if required.
Known Limitations
Test plan
waybionic_rviz_pluginson ROS 2 Jazzycolcon test --packages-select waybionic_rviz_pluginsengineer_view.launch.pyand verify mock normal/fault statescyclemode withuse_mock_diagnostics:=falseengineer_ar4_demo.launch.pyin an AR4-enabled workspace