chore: display hazard status that has no hardware_id

TomohitoAndo · TomohitoAndo · commit d331f5f55e61 · 2024-05-10T14:27:05.000+09:00
Signed-off-by: Tomohito Ando &lt;tomohito.ando@tier4.jp&gt;
diff --git a/common/tier4_system_rviz_plugin/src/mrm_summary_overlay_display.cpp b/common/tier4_system_rviz_plugin/src/mrm_summary_overlay_display.cpp
@@ -74,7 +74,7 @@ void insertNewlines(std::string & str, const size_t max_line_length)
 
 std::optional<std::string> generateMrmMessage(diagnostic_msgs::msg::DiagnosticStatus diag_status)
 {
-  if (diag_status.hardware_id == "" || diag_status.hardware_id == "system_error_monitor") {
+  if (diag_status.hardware_id == "system_error_monitor") {
     return std::nullopt;
   } else if (diag_status.level == diagnostic_msgs::msg::DiagnosticStatus::ERROR) {
     std::string msg = "- ERROR: " + diag_status.name + " (" + diag_status.message + ")";

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ void insertNewlines(std::string & str, const size_t max_line_length)`
`74`	`74`
`75`	`75`	`std::optional<std::string> generateMrmMessage(diagnostic_msgs::msg::DiagnosticStatus diag_status)`
`76`	`76`	`{`
`77`		`- if (diag_status.hardware_id == "" \|\| diag_status.hardware_id == "system_error_monitor") {`
	`77`	`+ if (diag_status.hardware_id == "system_error_monitor") {`
`78`	`78`	`return std::nullopt;`
`79`	`79`	`} else if (diag_status.level == diagnostic_msgs::msg::DiagnosticStatus::ERROR) {`
`80`	`80`	`std::string msg = "- ERROR: " + diag_status.name + " (" + diag_status.message + ")";`