mpl: throw an error if setClustersLocations is called with the incorrect number of clusters

joaomai · joaomai · commit 18d7cc4b2cb5 · 2025-09-11T17:27:20.000Z
Signed-off-by: João Mai &lt;jmai@precisioninno.com&gt;
diff --git a/src/mpl/src/SACoreSoftMacro.cpp b/src/mpl/src/SACoreSoftMacro.cpp
@@ -1020,7 +1020,10 @@ void SACoreSoftMacro::setClustersLocations(
     std::vector<std::pair<float, float>>& clusters_locations)
 {
   if (clusters_locations.size() != pos_seq_.size()) {
-    return;
+    logger_->error(MPL,
+                   52,
+                   "setClustersLocation called with a different numbers of "
+                   "clusters of that in the sequence pair");
   }
 
   for (int& id : pos_seq_) {

Original file line number	Diff line number	Diff line change
`@@ -1020,7 +1020,10 @@ void SACoreSoftMacro::setClustersLocations(`
`1020`	`1020`	`std::vector<std::pair<float, float>>& clusters_locations)`
`1021`	`1021`	`{`
`1022`	`1022`	`if (clusters_locations.size() != pos_seq_.size()) {`
`1023`		`- return;`
	`1023`	`+ logger_->error(MPL,`
	`1024`	`+ 52,`
	`1025`	`+ "setClustersLocation called with a different numbers of "`
	`1026`	`+ "clusters of that in the sequence pair");`
`1024`	`1027`	`}`
`1025`	`1028`
`1026`	`1029`	`for (int& id : pos_seq_) {`