Fix QoS setting to allow IPC#76
Conversation
clalancette
left a comment
There was a problem hiding this comment.
This looks very reasonable to me. The one change I'll ask for is a new launch test that exercises the intra-process comms.
To do that, you can look at the launch tests in e.g. https://github.com/ros/robot_state_publisher/blob/rolling/test/two_links_moving_joint-launch.py (which is exercised by https://github.com/ros/robot_state_publisher/blob/rolling/test/two_links_moving_joint-launch.py).
Thank you.
|
Hi @clalancette PS Is there documentation explaining how to create Google test units? I'm not experienced in this area |
|
@Myzhar @clalancette Are there any updates here? |
Remove default QoS settings and add QosOverridingOptions to allow the node to leverage IPC in composition
Fix #75