Ryan comments

Results 632 comments of


                                            Ryan

Fix LoadFromBag assumptions causing C++ exceptions during serialization (backport #438)

> @Ryanf55 How about adding if condition into vcs import block? To make it only import them `if $ROS_DISTRO==rolling`?

Fix LoadFromBag assumptions causing C++ exceptions during serialization (backport #438)

Interestingly, the `humble` branch doesn't contain that `vcs import` task. The failures here are from uncrustify, perhaps caused by a version difference between the binary in ubuntu 24 and the...

Fix LoadFromBag assumptions causing C++ exceptions during serialization (backport #438)

@wep21 Thoughts on trying to get this fixed, and once it's merged, we do another bugfix release for all the distros?

Fix LoadFromBag assumptions causing C++ exceptions during serialization (backport #438)

Rebased, fixed with `ament_uncrustify /home/ryan/Dev/grid_map_ws/src/grid_map/grid_map_ros --reformat`

Fix the README.md (Foxy --> Humble)

Can you change this to target rolling, and then say it's been tested on Humble and Jazzy?

fix(SpiralIterator): Avoid improper use of '++', which may cause prog…

@maximilianwulf Please merge this fix

fix(SpiralIterator): Avoid improper use of '++', which may cause prog…

This has been merged into all of the maintained ROS 2 branches.

SubmapIterator produces incorrect indices and segfaults? (ROS humble)

Hi, it's very possible you found a bug. Thank you for reporting it. If you can, do you mind forking grid_map, creating a branch, and showing the segfault in a...

AP_DDS: Added attitude target topic

Before merging this, we need to get upstream changes merged. I'd like the API-changing stuff to go in as a separate PR rather than bundling that with a new feature....

Access to code, being able to contribute/modify.

Ryan from the ArduPilot dev team here. I'm also interested in such an arrangement. Happy to sign an NDA or whatever is needed. STORM32 is awesome - would be great...