Ryan

Results 632 comments of Ryan

> ament_cpplint Done in 4381b27be282e80d9e97fe9171fbca388c8c550b No actions left other than dealing with transitive includes or not. Making the video now. Edit: No luck with the video, I tried a bunch...

Relates to https://github.com/mavlink/mavros/issues/2031

Latest CI: ``` E: Unable to locate package ros-rolling-octomap-server ERROR: the following rosdeps failed to install apt: command [apt-get install -y ros-rolling-octomap-server] failed ``` ``` docker run --net host -it...

Overall, this is looking great. Do you know what additional services would be required to get this call to work? ``` $ ros2 param set /ardupilot_dds LOIT_SPEED 1249 Wait for...

Needs a rebase on `master`.

> needs fix for stack frame size on build ``` ../../libraries/AP_DDS/AP_DDS_Client.cpp: In member function 'void AP_DDS_Client::on_request(uxrSession*, uxrObjectId, uint16_t, SampleIdentity*, ucdrBuffer*, uint16_t)': ../../libraries/AP_DDS/AP_DDS_Client.cpp:995:1: error: the frame size of 2800 bytes is...

> I tried changing [this file](https://github.com/PQuill33/ardupilot/blob/master/libraries/AP_DDS/wscript#L79) with container_prealloc_size = 8 changed to 2 and it can build fine. I tried to build Durandal and pixracer with that change and it...

Need to add a wiki. Don't use params if you are doing realtime control.

This conflicts with the established workflow here: https://ardupilot.org/dev/docs/ros2.html#installation-docker Consider testing that. That said, this doesn't hurt, so we could merge it.

> > This conflicts with the established workflow here: https://ardupilot.org/dev/docs/ros2.html#installation-docker > > Consider testing that. > > That said, this doesn't hurt, so we could merge it. > > I...