carconnectivity-connector-seatcupra 0.1a13__py3-none-any.whl → 0.1a15__py3-none-any.whl

{carconnectivity_connector_seatcupra-0.1a13.dist-info → carconnectivity_connector_seatcupra-0.1a15.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: carconnectivity-connector-seatcupra
-Version: 0.1a13
+Version: 0.1a15
 Summary: CarConnectivity connector for Seat and Cupra services
 Author: Till Steinbach
 License: MIT License
@@ -37,7 +37,7 @@ Classifier: Topic :: Software Development :: Libraries
 Requires-Python: >=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: carconnectivity>=0.4a6
+Requires-Dist: carconnectivity>=0.4a8
 Requires-Dist: oauthlib~=3.2.2
 Requires-Dist: requests~=2.32.3
 Requires-Dist: jwt~=1.3.1

{carconnectivity_connector_seatcupra-0.1a13.dist-info → carconnectivity_connector_seatcupra-0.1a15.dist-info}/RECORD

@@ -1,10 +1,10 @@
 carconnectivity_connectors/seatcupra/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-carconnectivity_connectors/seatcupra/_version.py,sha256=2nICb-m4-uY75MTKsmT4DgQRnEbdd7i9DdsoukRc60w,509
+carconnectivity_connectors/seatcupra/_version.py,sha256=C1vm3bxABjgmaD2Hpd79Ue7W_YMKggXDlANMpHBdVTI,509
 carconnectivity_connectors/seatcupra/capability.py,sha256=Oe9tC_u69bj6VmOuNJ21RKoETe2j3QyZCoz-VgcZPQ0,4523
 carconnectivity_connectors/seatcupra/charging.py,sha256=BJe_5GEB0JkP78tpU6kyKpwuwjDZHvm-kt3PTlpQHeU,3336
 carconnectivity_connectors/seatcupra/climatization.py,sha256=0xxWlxrheAPzkVT8WRQtbm6ExZmVdgW7lUdOXyS_qWY,1695
 carconnectivity_connectors/seatcupra/command_impl.py,sha256=mtw8ZwJLmf79fPDZ1N3ImLfB8Gt9JPbzjMuIo2y5v3M,2879
-carconnectivity_connectors/seatcupra/connector.py,sha256=UIorOzqnbnIwKr1IreCDDvFPB5L5g_4hdS3vspqv-Rk,100863
+carconnectivity_connectors/seatcupra/connector.py,sha256=ACP_TJitI1nQqxDzsiMn0HGdRAvoyY3NAZhpm8BltN4,101979
 carconnectivity_connectors/seatcupra/vehicle.py,sha256=s0G-HqG5qcwStDxD3649KgLMa3lKPZ4TOGWRJEuQzsQ,3403
 carconnectivity_connectors/seatcupra/auth/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 carconnectivity_connectors/seatcupra/auth/auth_util.py,sha256=Y81h8fGOMSMgPtE4wI_TI9WgE_s43uaPjRLBBINhj4g,4433
@@ -14,8 +14,8 @@ carconnectivity_connectors/seatcupra/auth/session_manager.py,sha256=ZIDvC848T3fy
 carconnectivity_connectors/seatcupra/auth/vw_web_session.py,sha256=CcI6m68IyRs6WsMDu-IsW3Dj85vyGiMmxvFqNETMHO0,10929
 carconnectivity_connectors/seatcupra/auth/helpers/blacklist_retry.py,sha256=f3wsiY5bpHDBxp7Va1Mv9nKJ4u3qnCHZZmDu78_AhMk,1251
 carconnectivity_connectors/seatcupra/ui/connector_ui.py,sha256=SNYnlcGJpbWhuLiIHD2l6H9IfSiMz3IgmvXsdossDnE,1412
-carconnectivity_connector_seatcupra-0.1a13.dist-info/LICENSE,sha256=PIwI1alwDyOfvEQHdGCm2u9uf_mGE8030xZDfun0xTo,1071
-carconnectivity_connector_seatcupra-0.1a13.dist-info/METADATA,sha256=OCSdxFdOXRtkkNDtzjWcDlcFGxRMTCQHQ8cq2AX-HzI,5641
-carconnectivity_connector_seatcupra-0.1a13.dist-info/WHEEL,sha256=jB7zZ3N9hIM9adW7qlTAyycLYW9npaWKLRzaoVcLKcM,91
-carconnectivity_connector_seatcupra-0.1a13.dist-info/top_level.txt,sha256=KqA8GviZsDH4PtmnwSQsz0HB_w-TWkeEHLIRNo5dTaI,27
-carconnectivity_connector_seatcupra-0.1a13.dist-info/RECORD,,
+carconnectivity_connector_seatcupra-0.1a15.dist-info/LICENSE,sha256=PIwI1alwDyOfvEQHdGCm2u9uf_mGE8030xZDfun0xTo,1071
+carconnectivity_connector_seatcupra-0.1a15.dist-info/METADATA,sha256=O10hztHSKmJMu8BqxiB6qh1isOsQv7gfOnU9M7vVffw,5641
+carconnectivity_connector_seatcupra-0.1a15.dist-info/WHEEL,sha256=jB7zZ3N9hIM9adW7qlTAyycLYW9npaWKLRzaoVcLKcM,91
+carconnectivity_connector_seatcupra-0.1a15.dist-info/top_level.txt,sha256=KqA8GviZsDH4PtmnwSQsz0HB_w-TWkeEHLIRNo5dTaI,27
+carconnectivity_connector_seatcupra-0.1a15.dist-info/RECORD,,

carconnectivity_connectors/seatcupra/_version.py

@@ -17,5 +17,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '0.1a13'
+__version__ = version = '0.1a15'
 __version_tuple__ = version_tuple = (0, 1)

carconnectivity_connectors/seatcupra/connector.py

@@ -6,6 +6,7 @@ import threading
 
 import json
 import os
+import traceback
 import logging
 import netrc
 from datetime import datetime, timezone, timedelta
@@ -83,6 +84,7 @@ class Connector(BaseConnector):
             if value is not None and value < timedelta(seconds=180):
                 raise ValueError('Intervall must be at least 180 seconds')
             return value
+        self._healthy = False
 
         self.interval._is_changeable = True  # pylint: disable=protected-access
         self.interval._add_on_set_hook(__check_interval)  # pylint: disable=protected-access
@@ -171,6 +173,7 @@ class Connector(BaseConnector):
         self._background_thread = threading.Thread(target=self._background_loop, daemon=False)
         self._background_thread.name = 'carconnectivity.connectors.seatcupra-background'
         self._background_thread.start()
+        self._healthy = True
 
     def _background_loop(self) -> None:
         self._stop_event.clear()
@@ -206,6 +209,10 @@ class Connector(BaseConnector):
             except TemporaryAuthenticationError as err:
                 LOG.error('Temporary authentification error during update (%s). Will try again after configured interval of %ss', str(err), interval)
                 self._stop_event.wait(interval)
+            except Exception as err:
+                LOG.critical('Critical error during update: %s', traceback.format_exc())
+                self._healthy = False
+                raise err
             else:
                 self._stop_event.wait(interval)
 
@@ -461,10 +468,10 @@ class Connector(BaseConnector):
                 else:
                     vehicle.connection_state._set_value(GenericVehicle.ConnectionState.UNKNOWN)  # pylint: disable=protected-access
                     LOG_API.info('Unknown connection state %s', vehicle_connection_data['connection']['mode'])
-                log_extra_keys(LOG_API, f'/api/v2/vehicles/{vin}/connection', vehicle_connection_data, {'connection'})
                 log_extra_keys(LOG_API, f'/api/v2/vehicles/{vin}/connection', vehicle_connection_data['connection'], {'mode'})
             else:
                 vehicle.connection_state._set_value(None)  # pylint: disable=protected-access
+            log_extra_keys(LOG_API, f'/api/v2/vehicles/{vin}/connection', vehicle_connection_data, {'connection'})
         else:
             vehicle.connection_state._set_value(None)  # pylint: disable=protected-access
 
@@ -861,6 +868,9 @@ class Connector(BaseConnector):
                 vehicle.climatization.settings.target_temperature._set_value(value=target_temperature,  # pylint: disable=protected-access
                                                                              measured=captured_at,
                                                                              unit=Temperature.C)
+                vehicle.climatization.settings.target_temperature.precision = 0.5
+                vehicle.climatization.settings.target_temperature.minimum = 16.0
+                vehicle.climatization.settings.target_temperature.maximum = 29.5
             elif 'targetTemperatureInFahrenheit' in data and data['targetTemperatureInFahrenheit'] is not None:
                 # pylint: disable-next=protected-access
                 vehicle.climatization.settings.target_temperature._add_on_set_hook(self.__on_air_conditioning_settings_change)
@@ -870,6 +880,9 @@ class Connector(BaseConnector):
                 vehicle.climatization.settings.target_temperature._set_value(value=target_temperature,  # pylint: disable=protected-access
                                                                              measured=captured_at,
                                                                              unit=Temperature.F)
+                vehicle.climatization.settings.target_temperature.precision = 0.5
+                vehicle.climatization.settings.target_temperature.minimum = 61.0
+                vehicle.climatization.settings.target_temperature.maximum = 85.5
             else:
                 vehicle.climatization.settings.target_temperature._set_value(None)  # pylint: disable=protected-access
             if 'climatisationWithoutExternalPower' in data and data['climatisationWithoutExternalPower'] is not None:
@@ -910,12 +923,12 @@ class Connector(BaseConnector):
             if data is not None:
                 if 'charging' in data and data['charging'] is not None:
                     if 'state' in data['charging'] and data['charging']['state'] is not None:
-                        if data['charging']['state'] in SeatCupraCharging.SeatCupraChargingState:
+                        if data['charging']['state'] in [item.value for item in SeatCupraCharging.SeatCupraChargingState]:
                             volkswagen_charging_state = SeatCupraCharging.SeatCupraChargingState(data['charging']['state'])
                             charging_state: Charging.ChargingState = mapping_seatcupra_charging_state[volkswagen_charging_state]
                         else:
                             LOG_API.info('Unkown charging state %s not in %s', data['charging']['state'],
-                                        str(SeatCupraCharging.SeatCupraChargingState))
+                                         str(SeatCupraCharging.SeatCupraChargingState))
                             charging_state = Charging.ChargingState.UNKNOWN
                         vehicle.charging.state._set_value(value=charging_state)  # pylint: disable=protected-access
                     else:
@@ -1162,12 +1175,15 @@ class Connector(BaseConnector):
             url = f'https://ola.prod.code.seat.cloud.vwgroup.com/v2/vehicles/{vin}/climatisation/start'
             if vehicle.climatization.settings is None:
                 raise CommandError('Could not control climatisation, there are no climatisation settings for the vehicle available.')
+            precision: float = 0.5
             if 'target_temperature' in command_arguments:
                 # Round target temperature to nearest 0.5
-                command_dict['targetTemperature'] = round(command_arguments['target_temperature'] * 2) / 2
+                command_dict['targetTemperature'] = round(command_arguments['target_temperature'] / precision) * precision
             elif vehicle.climatization.settings.target_temperature is not None and vehicle.climatization.settings.target_temperature.enabled \
                     and vehicle.climatization.settings.target_temperature.value is not None:
                 temperature_value = vehicle.climatization.settings.target_temperature.value
+                if vehicle.climatization.settings.target_temperature.precision is not None:
+                    precision = vehicle.climatization.settings.target_temperature.precision
                 if vehicle.climatization.settings.target_temperature.unit == Temperature.C:
                     command_dict['targetTemperatureUnit'] = 'celsius'
                 elif vehicle.climatization.settings.target_temperature.unit == Temperature.F:
@@ -1175,7 +1191,7 @@ class Connector(BaseConnector):
                 else:
                     command_dict['targetTemperatureUnit'] = 'celsius'
                 if temperature_value is not None:
-                    command_dict['targetTemperature'] = round(temperature_value * 2) / 2
+                    command_dict['targetTemperature'] = round(temperature_value / precision) * precision
             if 'target_temperature_unit' in command_arguments:
                 if command_arguments['target_temperature_unit'] == Temperature.C:
                     command_dict['targetTemperatureUnit'] = 'celsius'
@@ -1442,3 +1458,6 @@ class Connector(BaseConnector):
 
     def get_type(self) -> str:
         return "carconnectivity-connector-seatcupra"
+
+    def is_healthy(self) -> bool:
+        return self._healthy and super().is_healthy()