search

jinglemansweep / hacs-allpowers

Home Assistant custom component supporting Allpowers Batteries and Solar Generators
Apache License 2.0
14 stars 4 forks source link

IndexError: bytearray index out of range #4

Open skibbipl opened 6 months ago

skibbipl commented 6 months ago

I have Allpowers R600 power station. The integration seems to work, but HA logs a lot of errors:

Logger: root
Source: custom_components/allpowers_ble/allpowers.py:269
integration: Allpowers BLE Battery ([documentation](https://github.com/jinglemansweep/hacs-allpowers))
First occurred: 25 May 2024 at 09:29:29 (98066 occurrences)
Last logged: 17:20:22
A message handler raised an exception: bytearray index out of range

Traceback (most recent call last):
  File "src/dbus_fast/message_bus.py", line 802, in dbus_fast.message_bus.BaseMessageBus._process_message
  File "/usr/local/lib/python3.12/site-packages/bleak/backends/bluezdbus/manager.py", line 1008, in _parse_msg
    watcher.on_characteristic_value_changed(
  File "/usr/local/lib/python3.12/site-packages/bleak/backends/bluezdbus/client.py", line 177, in on_value_changed
    callback(bytearray(value))
  File "/config/custom_components/allpowers_ble/allpowers.py", line 269, in _notification_handler
    minutes_remaining = (256 * data[13]) + data[14]
                                           ~~~~^^^^
IndexError: bytearray index out of range