Ryan
Ryan
@khancyr Are you good with this PR now? If so, can we merge?
> Need rebase to get rid of merge commit and merge the two commit into one. And it is good to go ! Done. I force-pushed it.
Please leave it open. A maintainer will merge it shortly. I just tested the current state of this, and it works as intended.
Now that https://github.com/mavlink/qgroundcontrol/pull/13634 is merged, I'd like to discuss with the maintainers the right approach to support this. The new NTRIP manager class currently handles RTCM. MIssion planner's RTK GPS...
Caused by https://github.com/ArduPilot/ardupilot_wiki/pull/4642
A new binary release in humble should fix this.
Here's where some of our config settings are: https://github.com/ArduPilot/ardupilot/blob/master/libraries/AP_DDS/gen_config_h.py#L23
Using net=host and port 8888 is redundant, net=host opens everything. ufw is not installed by default on ubuntu. The errors explained are during configuration, not runtime, so I don't think...
Ok fair enough. Sounds like people generally ignore coding for a clean shutdown. Unless we wrap every call to rclcpp, such as `publish` with a check to rclcpp::ok or catching...