Skip to content

Conversation

soham2560
Copy link
Contributor

@soham2560 soham2560 commented Aug 20, 2025

Brief

This PR introduces a framework-managed feature for hardware components to publish their status via the control_msgs/msg/HardwareStatus message. This provides a standardized and thread-safe mechanism for hardware plugins to expose diagnostic information, enabled via a simple URDF parameter.

In this PR

  • Introduced configure_hardware_status_message and update_hardware_status_message virtual methods to HardwareComponentInterface.
  • The framework now automatically creates a publisher and a timer if a user's hardware plugin implements the configuration method.
  • Added Documentation

Copy link
Contributor

mergify bot commented Aug 20, 2025

This pull request is in conflict. Could you fix it @soham2560?

@soham2560 soham2560 marked this pull request as ready for review August 22, 2025 20:11
@soham2560
Copy link
Contributor Author

needs ros-controls/control_msgs#240

Copy link

codecov bot commented Aug 27, 2025

Codecov Report

❌ Patch coverage is 5.26316% with 36 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.01%. Comparing base (e3e8b9e) to head (de3391f).

Files with missing lines Patch % Lines
...ardware_interface/hardware_component_interface.hpp 5.26% 34 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2476      +/-   ##
==========================================
- Coverage   89.21%   89.01%   -0.21%     
==========================================
  Files         144      144              
  Lines       16334    16373      +39     
  Branches     1393     1397       +4     
==========================================
+ Hits        14573    14575       +2     
- Misses       1225     1260      +35     
- Partials      536      538       +2     
Flag Coverage Δ
unittests 89.01% <5.26%> (-0.21%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ardware_interface/hardware_component_interface.hpp 74.04% <5.26%> (-12.06%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant