Skip to content

343 - [Navigation] waypoint input#364

Open
Jelmerdw wants to merge 5 commits intomainfrom
343-navigation-waypoint-input
Open

343 - [Navigation] waypoint input#364
Jelmerdw wants to merge 5 commits intomainfrom
343-navigation-waypoint-input

Conversation

@Jelmerdw
Copy link
Collaborator

@Jelmerdw Jelmerdw commented Feb 2, 2026

Description

Added functionality for multiple waypoints, adding, removing, saving, loading and reordering.

Fixes: #343

Testing

uv run start.py panther_gps_navigation -v

Open rcdt_gui, place the waypoints as desired and start navigation.

Documentation

  • I have updated the documentation (if necessary)

Additional Notes

Any relevant screenshots, logs, or context.

@Jelmerdw Jelmerdw linked an issue Feb 2, 2026 that may be closed by this pull request
8 tasks
… it can be sorted (sorting not yet included in vehicle_control logic)

Signed-off-by: Rosalie <rosalie.van.ark@alliander.com>
@Jelmerdw Jelmerdw force-pushed the 343-navigation-waypoint-input branch from afcb453 to 8f86551 Compare February 2, 2026 12:36
Signed-off-by: Jelmer de Wolde <jelmer.de.wolde@alliander.com>
Signed-off-by: Jelmer de Wolde <jelmer.de.wolde@alliander.com>
Signed-off-by: Jelmer de Wolde <jelmer.de.wolde@alliander.com>
Signed-off-by: Jelmer de Wolde <jelmer.de.wolde@alliander.com>
@Jelmerdw Jelmerdw marked this pull request as ready for review February 3, 2026 15:02
e (events.GenericEventArguments): The event arguments.
"""
lat = e.args["latlng"]["lat"]
lng = e.args["latlng"]["lng"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

We're naming it long instead of lng somewhere above -- maybe good to pick one and keep it everywhere

@geurto
Copy link
Collaborator

geurto commented Feb 4, 2026

Is it possible to add a sprite indicating the robot's position on the map? It is sometimes difficult to gauge the exact robot position on the GUI.

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.

[Navigation] Waypoint input

3 participants