Merge pull request #1193 from merico-dev/1192-yaxismininterval-of-car…

…tesian-is-wrong-when-all-numbers-are-zero-and-group_by_key-is-set

1192 yaxismininterval of cartesian is wrong when all numbers are zero and group by key is set

api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devtable/api",
-  "version": "10.34.5",
+  "version": "10.34.6",
   "description": "",
   "main": "index.js",
   "scripts": {

dashboard/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devtable/dashboard",
-  "version": "10.34.5",
+  "version": "10.34.6",
   "license": "Apache-2.0",
   "publishConfig": {
     "access": "public",

dashboard/src/components/plugins/viz-components/cartesian/option/y-axis.ts

@@ -1,25 +1,41 @@
 import _ from 'lodash';
 import { ICartesianChartConf, IYAxisConf } from '../type';
 
-type SeriesData = number[];
+type SeriesDataOut = number[];
+type SeriesDataIn = [string | number, number][] | SeriesDataOut;
 type PartialSeriesConfType = {
   yAxisIndex: number;
-  data: SeriesData;
+  data: SeriesDataIn;
 };
 
+function getNumbersFromData(seriesData: SeriesDataIn): SeriesDataOut {
+  try {
+    if (Array.isArray(seriesData[0]) && seriesData[0]?.length >= 2) {
+      const s = seriesData as [string | number, number][];
+      return s.map((d) => Number(d[1]));
+    }
+
+    return [...seriesData] as SeriesDataOut;
+  } catch (error) {
+    console.error(error);
+    return [...seriesData] as SeriesDataOut;
+  }
+}
+
 function getReduceIntervalNeeds(series: PartialSeriesConfType[]) {
   if (series.length === 0) {
     return {};
   }
 
   const ret: Record<string, number> = {};
-  const datas: Record<number, SeriesData> = {};
+  const datas: Record<number, SeriesDataOut> = {};
   series.forEach((s) => {
+    const data = getNumbersFromData(s.data);
     if (!datas[s.yAxisIndex]) {
-      datas[s.yAxisIndex] = [...s.data];
+      datas[s.yAxisIndex] = data;
       return;
     }
-    datas[s.yAxisIndex] = datas[s.yAxisIndex].concat(s.data);
+    datas[s.yAxisIndex] = datas[s.yAxisIndex].concat(data);
   });
 
   Object.entries(datas).forEach(([index, data]) => {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devtable/root",
-  "version": "10.34.5",
+  "version": "10.34.6",
   "private": true,
   "workspaces": [
     "api",

settings-form/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devtable/settings-form",
-  "version": "10.34.5",
+  "version": "10.34.6",
   "license": "Apache-2.0",
   "publishConfig": {
     "access": "public",

website/package.json

@@ -2,7 +2,7 @@
   "name": "@devtable/website",
   "private": true,
   "license": "Apache-2.0",
-  "version": "10.34.5",
+  "version": "10.34.6",
   "scripts": {
     "dev": "vite",
     "preview": "vite preview"