@carto/api-client 0.5.8-alpha-others-orderby.1 → 0.5.8

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## 0.5
 
+## 0.5.8
+
+- feat(widgets): Add othersThreshold to WidgetSource.getCategories (#194)
+- perf(rasters): Improve performance of raster widget spatial filtering (#207)
+
 ### 0.5.7
 
 - feat(widgets): Allow local widget calculations without spatial filter (#204)

package/build/api-client.cjs

@@ -159,7 +159,6 @@ __export(src_exports, {
   getLayerProps: () => getLayerProps,
   getMaxMarkerSize: () => getMaxMarkerSize,
   getSizeAccessor: () => getSizeAccessor,
-  getSorter: () => getSorter,
   getSpatialIndexFromGeoColumn: () => getSpatialIndexFromGeoColumn,
   getTextAccessor: () => getTextAccessor,
   groupValuesByColumn: () => groupValuesByColumn,
@@ -5717,7 +5716,7 @@ function tileFeaturesRaster({
       options.spatialFilter
     );
     if (intersection3 === false) continue;
-    const tileSortedCells = cellToChildrenSorted(parent, cellResolution);
+    const tileSortedCells = cellToChildrenRaster(parent, cellResolution);
     for (let i = 0; i < tileSortedCells.length; i++) {
       if (intersection3 !== true && !intersection3.has(tileSortedCells[i])) {
         continue;
@@ -5745,17 +5744,19 @@ function isRasterTile(tile) {
 function isRasterTileVisible(tile) {
   return !!(tile.isVisible && tile.data?.cells?.numericProps);
 }
-function cellToChildrenSorted(parent, resolution) {
-  return (0, import_quadbin3.cellToChildren)(parent, resolution).sort(
-    (cellA, cellB) => {
-      const tileA = (0, import_quadbin3.cellToTile)(cellA);
-      const tileB = (0, import_quadbin3.cellToTile)(cellB);
-      if (tileA.y !== tileB.y) {
-        return tileA.y > tileB.y ? 1 : -1;
-      }
-      return tileA.x > tileB.x ? 1 : -1;
-    }
-  );
+function cellToChildrenRaster(parent, resolution) {
+  const parentTile = (0, import_quadbin3.cellToTile)(parent);
+  const childZ = Number(resolution);
+  const blockSize = 2 ** (childZ - parentTile.z);
+  const childBaseX = parentTile.x * blockSize;
+  const childBaseY = parentTile.y * blockSize;
+  const cells = [];
+  for (let i = 0, il = blockSize ** 2; i < il; i++) {
+    const x = childBaseX + i % blockSize;
+    const y = childBaseY + Math.floor(i / blockSize);
+    cells.push((0, import_quadbin3.tileToCell)({ x, y, z: childZ }));
+  }
+  return cells;
 }
 function isValidBandValue(value, nodata) {
   return Number.isNaN(value) ? false : nodata !== value;
@@ -6583,14 +6584,7 @@ var WidgetRemoteSource = class extends WidgetSource {
       rawResult,
       ...params
     } = options;
-    const {
-      column,
-      operation: operation2,
-      operationColumn,
-      operationExp,
-      othersThreshold,
-      orderBy
-    } = params;
+    const { column, operation: operation2, operationColumn, operationExp, othersThreshold } = params;
     if (operation2 === AggregationTypes.Custom) {
       assert2(operationExp, "operationExp is required for custom operation");
     }
@@ -6606,8 +6600,7 @@ var WidgetRemoteSource = class extends WidgetSource {
         operation: operation2,
         operationExp,
         operationColumn: operationColumn || column,
-        othersThreshold,
-        orderBy
+        othersThreshold
       },
       opts: { signal, headers: this.props.headers }
     });
@@ -7027,8 +7020,7 @@ function groupValuesByColumn({
   joinOperation,
   keysColumn,
   operation: operation2,
-  othersThreshold,
-  orderBy = "frequency_desc"
+  othersThreshold
 }) {
   if (Array.isArray(data) && data.length === 0) {
     return { rows: null };
@@ -7052,13 +7044,9 @@ function groupValuesByColumn({
   const allCategories = Array.from(groups).map(([name, value]) => ({
     name,
     value: targetOperation(value)
-  })).sort(getSorter(orderBy));
+  })).sort((a, b) => b.value - a.value);
   if (othersThreshold && allCategories.length > othersThreshold) {
     const otherValue = allCategories.slice(othersThreshold).flatMap(({ name }) => groups.get(name));
-    allCategories.push({
-      name: OTHERS_CATEGORY_NAME,
-      value: targetOperation(otherValue)
-    });
     return {
       rows: allCategories,
       metadata: {
@@ -7070,21 +7058,6 @@ function groupValuesByColumn({
     rows: allCategories
   };
 }
-function getSorter(orderBy) {
-  switch (orderBy) {
-    case "frequency_asc":
-      return (a, b) => a.value - b.value || localeCompare(a.name, b.name);
-    case "frequency_desc":
-      return (a, b) => b.value - a.value || localeCompare(a.name, b.name);
-    case "alphabetical_asc":
-      return (a, b) => localeCompare(a.name, b.name) || b.value - a.value;
-    case "alphabetical_desc":
-      return (a, b) => localeCompare(b.name, a.name) || b.value - a.value;
-  }
-}
-function localeCompare(a, b) {
-  return (a ?? "null").localeCompare(b ?? "null");
-}
 
 // src/operations/groupByDate.ts
 init_cjs_shims();
@@ -7659,7 +7632,6 @@ var WidgetTilesetSourceImpl = class extends WidgetSource {
     filterOwner,
     spatialFilter,
     othersThreshold,
-    orderBy = "frequency_desc",
     rawResult
   }) {
     const filteredFeatures = this._getFilteredFeatures(
@@ -7677,8 +7649,7 @@ var WidgetTilesetSourceImpl = class extends WidgetSource {
       joinOperation,
       keysColumn: column,
       operation: operation2,
-      othersThreshold,
-      orderBy
+      othersThreshold
     });
     if (rawResult) {
       return result;
@@ -10533,7 +10504,6 @@ function hashBuckets(initialCount) {
   getLayerProps,
   getMaxMarkerSize,
   getSizeAccessor,
-  getSorter,
   getSpatialIndexFromGeoColumn,
   getTextAccessor,
   groupValuesByColumn,