Skip to content

[rclcpp_components] Enable intra-process comm via LoadNode request.#871

Merged
hidmic merged 1 commit intomasterfrom
hidmic/intra-process-comm-for-components
Oct 10, 2019
Merged

[rclcpp_components] Enable intra-process comm via LoadNode request.#871
hidmic merged 1 commit intomasterfrom
hidmic/intra-process-comm-for-components

Conversation

@hidmic
Copy link
Copy Markdown
Contributor

@hidmic hidmic commented Sep 24, 2019

Look for a 'use_intra_process_comms' boolean in extra_arguments.

Look for a 'use_intra_process_comms' boolean in extra_arguments.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
@hidmic
Copy link
Copy Markdown
Contributor Author

hidmic commented Sep 24, 2019

We don't do much testing for rclcpp_components, do we? The most relevant one, test_component_manager_api , consistently fails and I bet it has to do with repeated node bring up / tear down.

I plan to test this manually, but until ros2/rcl#508 and friends are in ros2 component is broken.

@mjcarroll
Copy link
Copy Markdown
Member

I believe that it recently went from flaky to more consistently failing, I opened an issue here: #863

Copy link
Copy Markdown
Member

@mjcarroll mjcarroll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks Mich!

@hidmic
Copy link
Copy Markdown
Contributor Author

hidmic commented Oct 7, 2019

Running CI up to rclcpp_components:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • OSX Build Status
  • Windows Build Status

@hidmic hidmic merged commit 07cb443 into master Oct 10, 2019
@delete-merged-branch delete-merged-branch bot deleted the hidmic/intra-process-comm-for-components branch October 10, 2019 16:14
@iluetkeb
Copy link
Copy Markdown
Contributor

iluetkeb commented Nov 7, 2019

@hidmic Is there an example for how to use this in a launch file?

btw, I'm sure you guys must have plenty of example launch files that the community could have a look at, to figure out how to actually use launch properly. putting them somewhere would be awesome.

Edit: This is not in dashing, yet, right?

@ZhenshengLee
Copy link
Copy Markdown

@hidmic Is there an example for how to use this in a launch file?

btw, I'm sure you guys must have plenty of example launch files that the community could have a look at, to figure out how to actually use launch properly. putting them somewhere would be awesome.

@iluetkeb
the extra_arguments is here, define a ComposableNode with this parameter I guess.

https://github.com/ros2/launch_ros/blob/79f783b1fe5e06d71737e2955a8f4eb9c4f3923c/launch_ros/launch_ros/descriptions/composable_node.py#L39-L60

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants