Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Make diagnostic aggregator composable #321

Open
jlack1987 opened this issue Dec 5, 2023 · 1 comment
Open

Make diagnostic aggregator composable #321

jlack1987 opened this issue Dec 5, 2023 · 1 comment
Labels
enhancement This tackles a new feature of the code (and not a bug) PR welcome 💞 This issue has no PR that tries to implement it. Please create one! ros2 PR tackling a ROS2 branch

Comments

@jlack1987
Copy link

Should be a relatively easy feature to add to make the diagnostics aggregator composable so it can be composed with other nodes.
@ct2034
Copy link
Collaborator

ct2034 commented Dec 5, 2023

Hi @jlack1987

Thanks for bringing this up. I agree that this would be useful. Especially when paired with the node that is diagnosed, this would save some overhead.

Pull requests are always welcome 🤗

@ct2034 ct2034 added enhancement This tackles a new feature of the code (and not a bug) ros2 PR tackling a ROS2 branch labels Dec 5, 2023
@ct2034 ct2034 added the PR welcome 💞 This issue has no PR that tries to implement it. Please create one! label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement This tackles a new feature of the code (and not a bug) PR welcome 💞 This issue has no PR that tries to implement it. Please create one! ros2 PR tackling a ROS2 branch
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jlack1987 @ct2034