Now rounding computed min and max.

Author: ashleydavis

package.json

@@ -1,6 +1,6 @@
 {
     "name": "data-forge-plot",
-    "version": "0.3.3",
+    "version": "0.3.4",
     "description": "Plotting API for use with Data-Forge.",
     "main": "build/index.js",
     "types": "build/index.d.ts",

src/apply-defaults.ts

@@ -14,12 +14,12 @@ function extractValues(data: ISerializedDataFrame, seriesConfigs: IYAxisSeriesCo
     return flattened;
 }
 
-function findMin(values: number[]): number {
-    return Math.min(...values.filter(v => v !== undefined && v !== null && !Number.isNaN(v)));
+function computeMin(values: number[]): number {
+    return Math.floor(Math.min(...values.filter(v => v !== undefined && v !== null && !Number.isNaN(v))) * 100) / 100;
 }
 
-function findMax(values: number[]): number {
-    return Math.max(...values.filter(v => v !== undefined && v !== null && !Number.isNaN(v) && Number.isFinite(v)));
+function computeMax(values: number[]): number {
+    return Math.ceil(Math.max(...values.filter(v => v !== undefined && v !== null && !Number.isNaN(v) && Number.isFinite(v))) * 100) / 100;
 }
 
 //
@@ -87,7 +87,7 @@ export function applyDefaults(inputChartDef: IChartDef, plotDefaults?: IPlotConf
         y1Values = extractValues(chartDef.data, chartDef.axisMap.y);
 
         if (y1Values.length > 0) {        
-            chartDef.plotConfig.y.min = findMin(y1Values);
+            chartDef.plotConfig.y.min = computeMin(y1Values);
         }
     }
 
@@ -97,7 +97,7 @@ export function applyDefaults(inputChartDef: IChartDef, plotDefaults?: IPlotConf
         }
 
         if (y1Values.length > 0) {        
-            chartDef.plotConfig.y.max = findMax(y1Values);
+            chartDef.plotConfig.y.max = computeMax(y1Values);
         }
     }
 
@@ -110,7 +110,7 @@ export function applyDefaults(inputChartDef: IChartDef, plotDefaults?: IPlotConf
     if (chartDef.plotConfig.y2.min === undefined) {
         y2Values = extractValues(chartDef.data, chartDef.axisMap.y2);
         if (y2Values.length > 0) {
-            chartDef.plotConfig.y2.min = findMin(y2Values);
+            chartDef.plotConfig.y2.min = computeMin(y2Values);
         }
     }
 
@@ -120,7 +120,7 @@ export function applyDefaults(inputChartDef: IChartDef, plotDefaults?: IPlotConf
         }
 
         if (y2Values.length > 0) {
-            chartDef.plotConfig.y2.max = findMax(y2Values);
+            chartDef.plotConfig.y2.max = computeMax(y2Values);
         }
     }
 

src/test/apply-defaults.test.ts

@@ -464,4 +464,55 @@ describe("apply defaults", () => {
         expect(chartDef.plotConfig.y2!.min).toBeUndefined();
         expect(chartDef.plotConfig.y2!.max).toBeUndefined();
     });    
+
+    it("computed min and max are rounded", () => {
+
+        const testData = {
+            columnOrder: [ "a" ],
+            columns: {
+                a: "number",
+                b: "number",
+            },
+            index: {
+                type: "number",
+                values: [2, 3, 4],
+            },
+            values: [
+                {
+                    a: 10.123456,
+                },
+                {
+                    a: 20,
+                },
+                {
+                    a: 30.01234567,
+                },
+            ],
+        };
+    
+        const inputChartDef: any = {
+            data: testData,
+            plotConfig: {
+                y: {
+                },
+                y2: {
+                },
+            },
+            axisMap: {
+                y: [
+                    {
+                        series: "a",
+                    },
+                    {
+                        series: "b",
+                    },
+                ],
+            },
+        };
+
+        const chartDef = applyDefaults(inputChartDef);
+        expect(chartDef.plotConfig.y!.min).toBe(10.12);
+        expect(chartDef.plotConfig.y!.max).toBe(30.02);
+    });
+
 });