Handle derived data in cluster drop (#218)

alxvth · web-flow · commit dd545e1cd7ee · 2025-10-31T09:20:58.000+01:00
diff --git a/src/ScatterplotPlugin.cpp b/src/ScatterplotPlugin.cpp
@@ -248,7 +248,13 @@ ScatterplotPlugin::ScatterplotPlugin(const PluginFactory* factory) :
                             totalNumIndices += cluster.getIndices().size();
                         }
 
-                        if (totalNumIndices == _positionDataset->getNumPoints())
+                        int totalNumPoints = 0;
+                        if (_positionDataset->isDerivedData())
+                            totalNumPoints = _positionSourceDataset->getFullDataset<Points>()->getNumPoints();
+                        else
+                            totalNumPoints = _positionDataset->getFullDataset<Points>()->getNumPoints();
+
+                        if (totalNumIndices == totalNumPoints)
                         {
                             // Use the clusters set for points color
                             dropRegions << new DropWidget::DropRegion(this, "Color", description, "palette", true, [this, candidateDataset]() {

Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,13 @@ ScatterplotPlugin::ScatterplotPlugin(const PluginFactory* factory) :`
`248`	`248`	`totalNumIndices += cluster.getIndices().size();`
`249`	`249`	`}`
`250`	`250`
`251`		`- if (totalNumIndices == _positionDataset->getNumPoints())`
	`251`	`+ int totalNumPoints = 0;`
	`252`	`+ if (_positionDataset->isDerivedData())`
	`253`	`+ totalNumPoints = _positionSourceDataset->getFullDataset<Points>()->getNumPoints();`
	`254`	`+ else`
	`255`	`+ totalNumPoints = _positionDataset->getFullDataset<Points>()->getNumPoints();`
	`256`	`+`
	`257`	`+ if (totalNumIndices == totalNumPoints)`
`252`	`258`	`{`
`253`	`259`	`// Use the clusters set for points color`
`254`	`260`	`dropRegions << new DropWidget::DropRegion(this, "Color", description, "palette", true, [this, candidateDataset]() {`