Skip to content

Add support for events in subscriptions#13

Merged
ivanpauno merged 3 commits into
feature/eventsfrom
ivanpauno/support-events-in-subscriptions
Aug 24, 2020
Merged

Add support for events in subscriptions#13
ivanpauno merged 3 commits into
feature/eventsfrom
ivanpauno/support-events-in-subscriptions

Conversation

@ivanpauno
Copy link
Copy Markdown
Collaborator

@ivanpauno ivanpauno commented Aug 20, 2020

This PR implements event handlers for subscriptions.

I haven't added bindings for a subscription status yet, so I have not written tests.
Next PR is going to be that. After that, we can merge feature/events branch into foxy-devel.

Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
@ivanpauno ivanpauno added the enhancement New feature or request label Aug 20, 2020
@ivanpauno ivanpauno requested a review from jacobperron August 20, 2020 16:47
@ivanpauno ivanpauno self-assigned this Aug 20, 2020
Copy link
Copy Markdown

@jacobperron jacobperron left a comment

Choose a reason for hiding this comment

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

Minor comments. LGTM

Comment thread rcljava/src/main/java/org/ros2/rcljava/subscription/Subscription.java Outdated
Comment thread rcljava/src/main/java/org/ros2/rcljava/subscription/SubscriptionImpl.java Outdated
ivanpauno and others added 2 commits August 24, 2020 13:34
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>

Co-authored-by: Jacob Perron <jacob@openrobotics.org>
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>

Co-authored-by: Jacob Perron <jacob@openrobotics.org>
@ivanpauno ivanpauno merged commit 4fd64b3 into feature/events Aug 24, 2020
@ivanpauno ivanpauno deleted the ivanpauno/support-events-in-subscriptions branch August 24, 2020 16:56
ivanpauno added a commit that referenced this pull request Aug 31, 2020
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
ivanpauno added a commit that referenced this pull request May 17, 2021
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
ivanpauno added a commit to ros2-java/ros2_java that referenced this pull request Jan 14, 2022
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
ivanpauno added a commit to ros2-java/ros2_java that referenced this pull request Jan 25, 2022
Signed-off-by: Ivan Santiago Paunovic <ivanpauno@ekumenlabs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants