Skip to content

overload for node interfaces#700

Merged
Karsten1987 merged 3 commits intomasterfrom
fix_699
Apr 17, 2019
Merged

overload for node interfaces#700
Karsten1987 merged 3 commits intomasterfrom
fix_699

Conversation

@Karsten1987
Copy link
Copy Markdown
Contributor

@Karsten1987 Karsten1987 commented Apr 17, 2019

fixes #699

This PR allows to create an action server within a constructor inheriting from node and therefore composition of these nodes.

Signed-off-by: Karsten Knese karsten@openrobotics.org

Signed-off-by: Karsten Knese <karsten@openrobotics.org>
@Karsten1987 Karsten1987 added the in review Waiting for review (Kanban column) label Apr 17, 2019
@Karsten1987 Karsten1987 self-assigned this Apr 17, 2019
@Karsten1987
Copy link
Copy Markdown
Contributor Author

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

Signed-off-by: Karsten Knese <karsten@openrobotics.org>
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
@Karsten1987
Copy link
Copy Markdown
Contributor Author

Sorry @wjwwood, I've noticed too late that the action client follows the same principle. Hence, I've modified the PR for the client as well.

new round of CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@Karsten1987 Karsten1987 merged commit 60996d1 into master Apr 17, 2019
@delete-merged-branch delete-merged-branch bot deleted the fix_699 branch April 17, 2019 22:01
@Karsten1987 Karsten1987 removed the in review Waiting for review (Kanban column) label Apr 17, 2019
cho3 pushed a commit to cho3/rclcpp that referenced this pull request Jun 3, 2019
* overload for node interfaces

Signed-off-by: Karsten Knese <karsten@openrobotics.org>

* remove new line

Signed-off-by: Karsten Knese <karsten@openrobotics.org>

* overload client for node iterfaces

Signed-off-by: Karsten Knese <karsten@openrobotics.org>
DensoADAS pushed a commit to DensoADAS/rclcpp that referenced this pull request Aug 5, 2022
* Add time conversion logic diagrams to time control design

Signed-off-by: Emerson Knapp <emerson.b.knapp@gmail.com>
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.

create_actionserver can't be easily used with node interfaces

2 participants