SAIC-iSmart-API / saic-home-assistant-addon

SAIC-iSmart-API repository for Home Assistant OS (HASS.IO)
MIT License
58 stars 6 forks source link

Error since Update #104

Closed 00Twister00 closed 2 months ago

00Twister00 commented 2 months ago

hey i get this Error after Update Homeassistant

ValueError: non-hexadecimal number found in fromhex() arg at position 4 2024-08-09 11:06:24,241 [ INFO ] Connected to MQTT broker - mqtt_publisher 2024-08-09 11:06:24,258 [ INFO ] Scheduler started - apscheduler.scheduler Traceback (most recent call last): File "/usr/src/app/./mqtt_gateway.py", line 712, in asyncio.run(mqtt_gateway.run(), debug=debug_log_enabled()) File "/usr/local/lib/python3.11/asyncio/runners.py", line 190, in run return runner.run(main) ^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/asyncio/runners.py", line 118, in run return self._loop.run_until_complete(task) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete return future.result() ^^^^^^^^^^^^^^^ File "/usr/src/app/./mqtt_gateway.py", line 301, in run login_response_message = self.saic_api.login() ^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/site-packages/saic_ismart_client/saic_api.py", line 119, in login self.message_v1_1_coder.decode_response(login_response_hex, login_response_message) File "/usr/local/lib/python3.11/site-packages/saic_ismart_client/ota_v1_1/Message.py", line 13, in decode_response super().decode_response(message, decoded_message) File "/usr/local/lib/python3.11/site-packages/saic_ismart_client/common_model.py", line 401, in decode_response buffered_message_bytes = io.BytesIO(bytes.fromhex(message[5:])) ^^^^^^^^^^^^^^^^^^^^^^^^^^

dbrnz commented 2 months ago

Ditto...

dbrnz commented 2 months ago

Is this not the same as #51?

00Twister00 commented 2 months ago

I use the SAIC MQTT Gateway (Pyhton) (PreRelease) Version now ... Works fine

dbrnz commented 2 months ago

And I've since discovered that the SAIC URL was still pointing to the EU site when I was sure that I'd updated it to the AU one (this is with the legacy gateway). So all now good.

I suspect this issue can be closed as it appears to be due to trying to use the new SAIC API with legacy code.