Merge pull request #104 from cmu-delphi/dont_plot_nulls

dont plot null values

Author: RoniRos

src/components/Chart.svelte

@@ -719,10 +719,11 @@
     ctx.fillStyle = color;
     ctx.beginPath();
     for (let i = 0; i < data.length; i++) {
-      const y = value2y(dataset.getPointValue(i));
-      if (isNaN(y)) {
+      const ptVal = dataset.getPointValue(i);
+      if (isNaN(ptVal) || ptVal == null) {
         continue;
       }
+      const y = value2y(ptVal);
       date1 = date2;
       date2 = data[i].getDate().getIndex();
       if (date2 - date1 !== gap) {

src/data/DataSet.ts

@@ -65,7 +65,7 @@ export default class DataSet {
 
   getPointValue(index: number): number {
     const temp = this.data[index].getValue();
-    if (Number.isNaN(temp)) {
+    if (Number.isNaN(temp) || temp == null) {
       return temp;
     }
     return this.verticalOffset + temp * this.scale;