-
Notifications
You must be signed in to change notification settings - Fork 304
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
Add Topic Statistics Example #281
Add Topic Statistics Example #281
Conversation
Fix linting errors Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>
PTAL @jacobperron @hidmic |
@jacobperron @hidmic friendly ping :-) |
rclcpp/topics/minimal_subscriber/member_function_with_topic_statistics.cpp
Outdated
Show resolved
Hide resolved
Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>
10de0dc
to
981f86c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM after fixing the uncrustify error. (I think the PR CI job should be enough, no need to rerun CI on all platforms).
Signed-off-by: Devin Bonnie <dbbonnie@amazon.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Adds an example subscriber which enables topic statistics and illustrates some of the configuration options.
Related to ros2/ros2_documentation#824, https://github.com/ros-tooling/aws-roadmap/issues/246
Signed-off-by: Devin Bonnie dbbonnie@amazon.com