Skip to content

Add waypoint radius input to missions page#1184

Open
1Blademaster wants to merge 4 commits into
mainfrom
1182-feature-display-waypoint-radius-on-missions-page
Open

Add waypoint radius input to missions page#1184
1Blademaster wants to merge 4 commits into
mainfrom
1182-feature-display-waypoint-radius-on-missions-page

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 11, 2026 18:46
@1Blademaster 1Blademaster linked an issue May 11, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a waypoint radius control to the GCS Missions page and wires it through Redux + Socket.IO to update the corresponding autopilot parameter, also visualizing the radius around mission waypoints on the map.

Changes:

  • Add set_waypoint_radius Socket.IO endpoint on the radio side that selects the correct parameter name based on vehicle type and firmware.
  • Add a Redux emitter/action to send waypoint radius updates from the GCS to the radio.
  • Add a Missions-page NumberInput for waypoint radius and render waypoint-radius rings on the mission map.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
radio/app/endpoints/nav.py Adds server-side waypoint radius setter with param-name selection by vehicle type/firmware.
gcs/src/redux/slices/paramsSlice.js Adds emitSetWaypointRadius action to drive Socket.IO emission.
gcs/src/redux/middleware/emitters.js Emits set_waypoint_radius over Socket.IO when the new action fires.
gcs/src/missions.jsx Adds debounced waypoint radius input and passes value into the map.
gcs/src/components/missions/missionsMap.jsx Threads waypointRadius prop into the map’s MissionItems rendering.
gcs/src/components/mapComponents/missionItems.jsx Draws radius rings (GeoJSON + map layer) around displayed mission items.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread gcs/src/components/mapComponents/missionItems.jsx Outdated
Comment thread gcs/src/missions.jsx Outdated
Comment thread gcs/src/missions.jsx Outdated
Comment thread gcs/src/redux/middleware/emitters.js Outdated
Comment thread gcs/src/components/mapComponents/missionItems.jsx Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Comment thread radio/app/controllers/navController.py Outdated
Comment thread radio/app/controllers/navController.py
Comment thread radio/tests/test_nav.py Outdated
Comment thread radio/tests/test_nav.py
Comment thread gcs/src/redux/slices/paramsSlice.js
Comment thread radio/app/endpoints/nav.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Display waypoint radius on missions page

2 participants