Exclude rowid from filters for poly chart vega query only (#679)

heavyai · Oct 25, 2024 · 740f6a9 · 740f6a9
1 parent 7a13604
commit 740f6a9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/dist/charting.js b/dist/charting.js
@@ -66233,7 +66233,9 @@ function rasterLayerPolyMixin(_layer) {
     }
     var allFilters = _layer.crossfilter().getFilter(layerName);
     var otherChartFilters = allFilters.filter(function (f, i) {
-      return !_layer.dimension() || i !== _layer.dimension().getDimensionIndex() && f !== "" && f !== null;
+      return !_layer.dimension() || i !== _layer.dimension().getDimensionIndex() && f !== "" && f !== null &&
+      // Kinda hacky, we don't want to include rowId filter in vega query, but we want everything else
+      !f.includes("rowid");
     });
     var firstElem = true;
     otherChartFilters.forEach(function (value) {

diff --git a/src/mixins/raster-layer-poly-mixin.js b/src/mixins/raster-layer-poly-mixin.js
@@ -807,7 +807,11 @@ export default function rasterLayerPolyMixin(_layer) {
     const otherChartFilters = allFilters.filter(
       (f, i) =>
         !_layer.dimension() ||
-        (i !== _layer.dimension().getDimensionIndex() && f !== "" && f !== null)
+        (i !== _layer.dimension().getDimensionIndex() &&
+          f !== "" &&
+          f !== null &&
+          // Kinda hacky, we don't want to include rowId filter in vega query, but we want everything else
+          !f.includes("rowid"))
     )
 
     let firstElem = true