possible fix for #16

setup.py

@@ -1,7 +1,7 @@
 from setuptools import setup
 setup(
     name='teslajsonpy',
-    version='0.0.22',
+    version='0.0.24',
     packages=['teslajsonpy'],
     include_package_data=True,
     python_requires='>=3',
@@ -17,6 +17,8 @@
         'Programming Language :: Python :: 3',
         'Programming Language :: Python :: 3.4',
         'Programming Language :: Python :: 3.5',
+        'Programming Language :: Python :: 3.6',
+        'Programming Language :: Python :: 3.7',
         'Topic :: Internet',
     ],
 )

teslajsonpy/Exceptions.py

@@ -19,3 +19,6 @@ def __init__(self, code,  *args, **kwargs):
             self.message = 'SERVICE_MAINTENANCE'
         elif self.code > 299:
             self.message = "UNKNOWN_ERROR"
+
+class RetryLimitError(TeslaException):
+    pass

teslajsonpy/connection.py

@@ -9,6 +9,7 @@
 
 class Connection(object):
     """Connection to Tesla Motors API"""
+
     def __init__(self, email, password):
         """Initialize connection object"""
         self.user_agent = 'Model S 2.1.79 (SM-G900V; Android REL 4.4.4; en_US'

teslajsonpy/controller.py

@@ -7,6 +7,8 @@
 from teslajsonpy.BinarySensor import ParkingSensor, ChargerConnectionSensor
 from teslajsonpy.Charger import ChargerSwitch, RangeSwitch
 from teslajsonpy.GPS import GPS, Odometer
+from functools import wraps
+from .Exceptions import RetryLimitError
 
 
 class Controller:
@@ -21,11 +23,17 @@ def __init__(self, email, password, update_interval):
         self.__driving = {}
         self.__gui = {}
         self.__last_update_time = {}
+        self.__last_wake_up_time = {}
         self.__lock = RLock()
+        self.__car_online = {}
+
         cars = self.__connection.get('vehicles')['response']
+
         for car in cars:
             self.__last_update_time[car['id']] = 0
+            self.__last_wake_up_time[car['id']] = 0
             self.__update[car['id']] = True
+            self.__car_online[car['id']] = False
             self.update(car['id'])
             self.__vehicles.append(Climate(car, self))
             self.__vehicles.append(Battery(car, self))
@@ -55,15 +63,37 @@ def command(self, vehicle_id, name, data={}):
     def list_vehicles(self):
         return self.__vehicles
 
-    def wake_up(self, vehicle_id):
-        self.post(vehicle_id, 'wake_up')
-
+    def wake_up(f):
+        @wraps(f)
+        def wrapped(inst, *args, **kwargs):
+            retries = 0
+            sleep_delay = 2
+            while True:
+                result = inst._wake_up(*args)
+                if not result:
+                    if retries < 5:
+                        time.sleep(sleep_delay)
+                        continue
+                    else:
+                        raise RetryLimitError
+                else:
+                    break
+            return f(inst, *args, **kwargs)
+        return wrapped
+
+    def _wake_up(self, vehicle_id):
+        cur_time = int(time.time())
+        if not self.__car_online[vehicle_id] or cur_time - self.__last_wake_up_time[vehicle_id] > 300:
+            self.__car_online[vehicle_id] = self.post(vehicle_id, 'wake_up')['response']['state'] == 'online'
+            self.__last_wake_up_time[vehicle_id] = cur_time
+        return self.__car_online[vehicle_id]
+
+    @wake_up
     def update(self, car_id):
         cur_time = time.time()
         with self.__lock:
             if (self.__update[car_id] and
              (cur_time - self.__last_update_time[car_id] > self.update_interval)):
-                self.wake_up(car_id)
                 data = self.get(car_id, 'data')
                 if data and data['response']:
                     self.__climate[car_id] = data['response']['climate_state']
@@ -79,26 +109,33 @@ def update(self, car_id):
                     self.__driving[car_id] = False
                     self.__gui[car_id] = False
 
+    @wake_up
     def get_climate_params(self, car_id):
         return self.__climate[car_id]
 
+    @wake_up
     def get_charging_params(self, car_id):
         return self.__charging[car_id]
 
+    @wake_up
     def get_state_params(self, car_id):
         return self.__state[car_id]
 
+    @wake_up
     def get_drive_params(self, car_id):
         return self.__driving[car_id]
 
+    @wake_up
     def get_gui_params(self, car_id):
         return self.__gui[car_id]
 
+    @wake_up
     def get_updates(self, car_id=None):
         if car_id is not None:
             return self.__update[car_id]
         else:
             return self.__update
 
+    @wake_up
     def set_updates(self, car_id, value):
         self.__update[car_id] = value