Skip to content

feat: add publish_values parameter#418

Closed
MrBlenny wants to merge 8 commits into
ros:ros2from
greenroom-robotics:feat/add-publish-values
Closed

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

Conversation

@MrBlenny
Copy link
Copy Markdown
Contributor

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
@ct2034 ct2034 mentioned this pull request May 20, 2026
@ct2034
Copy link
Copy Markdown
Collaborator

ct2034 commented May 20, 2026

merged via #597

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