Improve live diagnostics messages

[aj0khi]

just checking if my work actually sent yassine

Summary by CodeRabbit

• New Features
  • Diagnostic statuses now include descriptive messages for board temperature, motor current, and IMU orientation readings.
  • Diagnostic alerts now display their messages even when the status is normal or marked as “OK.”

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3770a296-34e5-44cf-8464-b44257038ecf

📥 Commits

Reviewing files that changed from the base of the PR and between 02d0b90 and ba315fd.

📒 Files selected for processing (2)

• waybionic_rviz_plugins/scripts/temporary_diagnostics_publisher.py
• waybionic_rviz_plugins/src/ros_diagnostics_source.cpp

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

The diagnostic publisher now adds descriptive messages for five normal statuses. ROS diagnostic conversion now includes non-empty messages in alerts for OK statuses.

Changes

Diagnostic message propagation

Layer / File(s)	Summary
Message generation and alert conversion waybionic_rviz_plugins/scripts/temporary_diagnostics_publisher.py, waybionic_rviz_plugins/src/ros_diagnostics_source.cpp	Normal board temperature, motor current, and IMU statuses now include descriptive messages. Diagnostic alerts now retain non-empty messages for all normalized statuses, including Ok.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to ba315

The PR makes a small, localized diagnostics-message change, and no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: yassinsolim

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: improving live diagnostic messages.
Docstring Coverage	✅ Passed	Docstring check was indeterminate for this PR — some files could not be analyzed in time. Not blocking.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/diagnostics-cli-ajmal

---

_{Comment @coderabbitai help to get the list of available commands.}