Skip to content

Merge ROS2 SLAM/NAV integration branch to develop - #236

Open
jonpg01 wants to merge 13 commits into
developfrom
feature/ros2_slam_nav_int
Open

Merge ROS2 SLAM/NAV integration branch to develop#236
jonpg01 wants to merge 13 commits into
developfrom
feature/ros2_slam_nav_int

Conversation

@jonpg01

@jonpg01 jonpg01 commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

stefanvogel and others added 13 commits April 14, 2026 10:45
* fix(APPRemoteControl): add ROBOT_SET channel and fix initial pose handling
* chore: remove default PIO envs
* docs: fix typos
* fix(APPRemoteControl): improve logs
webots-controller crashes with SIGSEGV (exit 139) in libController.so
when the remote Webots host closes the TCP connection. Replace the raw
shell action with a subprocess wrapper that treats exit code 139 as
success, since the DCS process itself exits cleanly at that point.

Also replace `return status` with `exit(status)` in main() to bypass
C++ static destructor teardown: ~webots::Robot() calls
wb_robot_cleanup() and ~MqttClient() calls mosquitto_loop_stop(),
both of which crash or block when the Webots connection is already gone.
@jonpg01
jonpg01 requested a review from stefanvogel August 18, 2026 08:26
@stefanvogel stefanvogel changed the title Feature/ros2 slam nav int Merge ROS2 SLAM/NAV integration branch to develop Aug 18, 2026
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.

2 participants