Fix handling of nulls in auto-axis

Mechanical-Advantage · Jan 24, 2022 · 218cf95 · 218cf95
1 parent 8c71a59
commit 218cf95
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 10 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1 +1 @@
-Removed superfluous features to simplify code base.
+Fixed handling of `null` values when calculating auto-axis range.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
   "name": "advantage-scope",
   "productName": "Advantage Scope",
-  "version": "1.13.1",
+  "version": "1.13.2",
   "description": "Logging tool from FRC Team 6328.",
   "main": "main.js",
   "scripts": {

diff --git a/www/modules/tabControllers/lineGraphController.mjs b/www/modules/tabControllers/lineGraphController.mjs
@@ -571,6 +571,7 @@ export class LineGraphController {
       function arrayMin(arr) {
         var len = arr.length, min = Infinity;
         while (len--) {
+          if (arr[len] == null) continue
           if (arr[len] < min) {
             min = arr[len];
           }
@@ -581,6 +582,7 @@ export class LineGraphController {
       function arrayMax(arr) {
         var len = arr.length, max = -Infinity;
         while (len--) {
+          if (arr[len] == null) continue
           if (arr[len] > max) {
             max = arr[len];
           }
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		Removed superfluous features to simplify code base.
		Fixed handling of `null` values when calculating auto-axis range.