Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

overload for node interfaces #700

Merged
merged 3 commits into from
Apr 17, 2019
Merged

overload for node interfaces #700

merged 3 commits into from
Apr 17, 2019

Conversation

Karsten1987
Copy link
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
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
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