Skip to content

Methods to retrieve matched counts on publisher and subscriber#155

Merged
mjcarroll merged 4 commits intomasterfrom
subscriber_status_callback
Nov 28, 2018
Merged

Methods to retrieve matched counts on publisher and subscriber#155
mjcarroll merged 4 commits intomasterfrom
subscriber_status_callback

Conversation

@mjcarroll
Copy link
Copy Markdown
Member

@mjcarroll mjcarroll commented Nov 12, 2018

This provides an API for retrieving the number of matched publishers to a subscription and vice-versa. It is a method to be used with forthcoming additions in creating a port of SubscriberStatusCallback (from ROS 1, see: https://docs.ros.org/api/roscpp/html/namespaceros.html#ac61b99a2913cd421fb915395a67e71a1) for ROS2.

This is different from the existing graph methods, as it should interact directly with a publisher or subscription to retrieve the matched count.

Requires changes in:

Copy link
Copy Markdown
Member

@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.

Other than two typos, LGTM

@mjcarroll
Copy link
Copy Markdown
Member Author

CI:

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

@mjcarroll mjcarroll added in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Nov 27, 2018
Copy link
Copy Markdown
Member

@wjwwood wjwwood left a comment

Choose a reason for hiding this comment

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

lgtm

Co-Authored-By: mjcarroll <michael@openrobotics.org>
@mjcarroll
Copy link
Copy Markdown
Member Author

Rerun CI after all changes:

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

@mjcarroll
Copy link
Copy Markdown
Member Author

Once more with rebased rmw_fastrtps:

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

@mjcarroll mjcarroll merged commit c3b7077 into master Nov 28, 2018
@mjcarroll mjcarroll removed the in review Waiting for review (Kanban column) label Nov 28, 2018
@wjwwood wjwwood deleted the subscriber_status_callback branch December 19, 2018 19:59
dabonnie pushed a commit to aws-ros-dev/rmw that referenced this pull request Apr 2, 2019
)

* Methods to retrieve matched counts on pub/sub.

* Address reviewer feedback with naming and docs.

* Fix typos.

* Apply suggestions from code review

Co-Authored-By: mjcarroll <michael@openrobotics.org>
Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>
@AndreasR30
Copy link
Copy Markdown

This pull request enables the feature getNumSubscribers() on a publisher but it does not enable your mentioned ROS1 feature SubscriberStatusCallback. Is there any chance to get this feature back in ROS2?

@fujitatomoya
Copy link
Copy Markdown
Collaborator

@AndreasR30 probably #331 this is what you need? you are very much welcome for the review!

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.

6 participants