afrixs

Migration guide recommends to throw exceptions in service callbacks

1

[Migration guide says](https://docs.ros.org/en/rolling/Contributing/Migration-Guide.html#use-of-service-objects) that "Service callbacks in ROS 2 do not have boolean return values. Instead of returning false on failures, throwing exceptions is recommended." However throwing an exception makes...

Shared memory not working with transient_local durability

11

## Bug report **Required Info:** - Operating System: - Ubuntu 22.04.1 - Installation type: - binaries - Version or commit hash: - 1.4.0-1jammy.20220711.152242 - DDS implementation: - CycloneDDS - Client...

loadFromBag crashes in ROS2 when bag contains multiple topics

3

I guess that `topic` parameter was meant to be used, e.g. ``` if (bag_message->topic_name != topic) { continue; } ``` could be added [here](https://github.com/ANYbotics/grid_map/blob/1457713d683dbf2c0e12a15f0d40b61db4170b66/grid_map_ros/src/GridMapRosConverter.cpp#L726)

Crash when namespace is longer than topic name

3

`ros-iron-image-transport-plugins` version: `3.1.0-1jammy.20230908.194115` My node uses ``` image_transport::create_subscription(this, "input", [&](auto &msg) {}, "compressed", rmw_qos_profile_sensor_data); ``` When I run the node in a longer namespace, e.g. `camera360`, it crashes at [this...

There was recently a backporting update of 3.8 where [this line](https://github.com/BehaviorTree/BehaviorTree.CPP/blame/95210b01f12f30fc25946c03db0fed0b826be1fd/src/controls/reactive_sequence.cpp#L31) changed from ``` for (size_t i = index + 1; i < childrenCount(); i++) ``` to ``` for (size_t...

This fixes #6 and #7

Another thing that surprised me in the use-case described in #6 was that the first iteration was performed although `num_iterations_after_valid` was set to 0. I think that initial trajectory, if...

This bug is best visible for very short trajectories (i.e. difference between `first` and `last` parameters is less than 1e-4). In this case robot jerks back and forth a bit...

[Don’t Merge] Sync with MoveIt1

8

### Description Syncing the following commits ``` * [2022-09-09][b3b363f7b] | Limit Cartesian speed for link(s) (#2856) {{Michael Görner}} * [2022-09-09][87038c3e2] | Update panda_moveit_config to noetic-devel in .rosinstall files {{Robert Haschke}}...