Skip to content

feat: add publish_values parameter#418

Open
MrBlenny wants to merge 8 commits intoros:ros2from
greenroom-robotics:feat/add-publish-values
Open

feat: add publish_values parameter#418
MrBlenny wants to merge 8 commits intoros:ros2from
greenroom-robotics:feat/add-publish-values

Conversation

@MrBlenny
Copy link
Copy Markdown

When operating under bandwidth-constrained conditions it useful to have the option to turn off the values in the DiagnosticStatus message. These contain lots of redundant information in the parent diagnostics.

@MrBlenny MrBlenny marked this pull request as ready for review December 17, 2024 06:11
@ct2034 ct2034 added the ros2 PR tackling a ROS2 branch label Jan 7, 2025
@ct2034 ct2034 self-assigned this Jan 7, 2025
@ct2034 ct2034 added the enhancement This tackles a new feature of the code (and not a bug) label Feb 10, 2025
Copy link
Copy Markdown
Collaborator

@ct2034 ct2034 left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution. Can you please add a better explanation of that the published values are.

@ct2034 ct2034 added the needs more work Someone has worked on this but more work is needed label May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement This tackles a new feature of the code (and not a bug) needs more work Someone has worked on this but more work is needed ros2 PR tackling a ROS2 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants