From c14fa299e1bfbce45b682d552f0da1ea0e82f22b Mon Sep 17 00:00:00 2001
From: jamespcole <jamespcole@gmail.com>
Date: Sun, 5 Apr 2015 00:49:52 +1100
Subject: [PATCH] Improved the efficiency and accuracy of the line graphs

---
 .../polymer/components/state-timeline.html    | 34 ++++++++++++-------
 1 file changed, 21 insertions(+), 13 deletions(-)

diff --git a/homeassistant/components/frontend/www_static/polymer/components/state-timeline.html b/homeassistant/components/frontend/www_static/polymer/components/state-timeline.html
index 506839820ab..a5b017628e1 100644
--- a/homeassistant/components/frontend/www_static/polymer/components/state-timeline.html
+++ b/homeassistant/components/frontend/www_static/polymer/components/state-timeline.html
@@ -155,17 +155,21 @@
             },
             hAxis: {
               format: 'H:mm'
-            }
+            },
+            chartArea:{left:'60',width:"95%"}
           };
 
         if(isSingleDevice) {
           options.legend.position = 'none';
         }
+        //var times = _(deviceStates).chain().flatten().pluck('lastChangedAsDate').unique().value();
+        var times = _.pluck(_.flatten(deviceStates), "lastChangedAsDate");
+        times = _.uniq(times, function(e) {
+            return e.getTime();
+          });
 
-        var times = _(deviceStates).chain().flatten().pluck('lastChangedAsDate').unique().value();
+        times = _.sortBy(times, function(o) { return o; });
 
-        times = _.sortBy(times, function(o) { return o; })
-        //console.log(times);
         var data = [];
         var empty = new Array(deviceStates.length);
         for(var i = 0; i < empty.length; i++) {
@@ -181,18 +185,23 @@
           currentDate.setMinutes(prevDate.getMinutes() + 1);
           if(currentDate >= times[timeIndex] && timeIndex < times.length) {
             //this is so we don't get jumps in the graph
-            var beforePoint = new Date(times[timeIndex]);
-            beforePoint.setSeconds(times[timeIndex].getSeconds() - 1)
-            data.push([beforePoint].concat(empty));
+            if(times[timeIndex].getTime() - prevDate.getTime() > 30 * 1000) {
+              var beforePoint = new Date(times[timeIndex].getTime() - 2000);
+              data.push([beforePoint].concat(empty));
+            }
+
             data.push([times[timeIndex]].concat(empty));
+            prevDate = times[timeIndex];
             timeIndex++;
           }
-          //else {
-            //data.push([currentDate].concat(empty));
-          //}
+          else {
+            prevDate = currentDate;
+          }
+
 
-          prevDate = currentDate;
         }
+        data.push([endDate].concat(empty));
+
 
         var deviceCount = 0;
         deviceStates.forEach(function(device) {
@@ -207,13 +216,12 @@
 
             currentState = state.state;
             var start = state.lastChangedAsDate;
-            //console.log(start);
             if(state.state == 'None') {
               currentState = previousState;
             }
             for(var i = lastIndex; i < data.length; i++) {
               data[i][1 + deviceCount] = parseFloat(previousState);
-              if(data[i][0] == start) {
+              if(data[i][0].getTime() == start.getTime()) {
                 data[i][1 + deviceCount] = parseFloat(currentState);
                 lastIndex = i;
                 break;