Merge pull request #1426 from tier4/fix/remove-topic-logic

hakuturu583 · web-flow · commit 742a49958ed2 · 2024-11-01T17:43:34.000+09:00
Fixed logic to remove duplicate topics.
diff --git a/rviz_plugins/openscenario_visualization/src/context_panel_plugin.cpp b/rviz_plugins/openscenario_visualization/src/context_panel_plugin.cpp
@@ -111,7 +111,10 @@ void ContextPanel::updateTopicCandidates()
         }
       }
     }
-    std::unique(topics_.begin(), topics_.end());
+
+    std::set<std::string> unique_topics(topics_.begin(), topics_.end());
+    topics_.assign(unique_topics.begin(), unique_topics.end());
+
     ui_->TopicSelect->clear();
     ui_->TopicSelect->addItem("simulation/context");
     for (const auto & topic : topics_) {

Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,10 @@ void ContextPanel::updateTopicCandidates()`
`111`	`111`	`}`
`112`	`112`	`}`
`113`	`113`	`}`
`114`		`- std::unique(topics_.begin(), topics_.end());`
	`114`	`+`
	`115`	`+ std::set<std::string> unique_topics(topics_.begin(), topics_.end());`
	`116`	`+ topics_.assign(unique_topics.begin(), unique_topics.end());`
	`117`	`+`
`115`	`118`	`ui_->TopicSelect->clear();`
`116`	`119`	`ui_->TopicSelect->addItem("simulation/context");`
`117`	`120`	`for (const auto & topic : topics_) {`