alemuro / beewi_smartclim Goto Github PK

• This library supports an integration in Home Assistant.
• The upcoming pip resolver will be more strict in resolving package version conflicts:
  http://pyfound.blogspot.com/2020/03/new-pip-resolver-to-roll-out-this-year.html#what-will-change
• To avoid future issues it would be good if this library would change its pinning to allow any higher version and just set a minimum version.

Thanks for your support of Home Assistant!

Hello,
I've activated beewi_smartclim in my Home Assistant setup (containers on Raspberry Pi).
I'm running Home Assistant 0.118.0 which seems to embed btlewrap 0.1.0 and beewi_smartclim 0.0.7 (even if this version seems to require btlewrap 0.0.8).

After that, sensor states were "unknown" in home assistant and in HA Core Logs, I had following kind of error :

2020-11-19 21:04:21 ERROR (MainThread) [homeassistant.helpers.entity] Update for sensor.capteur_beewi_battery fails
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 278, in async_update_ha_state
    await self.async_device_update()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 474, in async_device_update
    raise exc
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/homeassistant/homeassistant/components/beewi_smartclim/sensor.py", line 98, in update
    self._poller.update_sensor()
  File "/usr/local/lib/python3.8/site-packages/beewi_smartclim/__init__.py", line 72, in update_sensor
    bt_interface = BluetoothInterface(backend=self._backend, adapter="hci0")
TypeError: __init__() takes 2 positional arguments but 3 were given

I think this problem should also arise with btlewrap 0.0.8 since BluetoothInterface __ init__ prototype seems not to have changed between 0.0.8 and 0.1.0.
I've solved (to be confirmed !) the error by changing call to BluetoothInterface in beewi_smartclim's __ init__.py file :
bt_interface = BluetoothInterface(self._backend, adapter="hci0")
changed to :
bt_interface = BluetoothInterface(backend=self._backend, adapter="hci0")

Maybe it could be corrected inside beewi_smartclim ?

By the way, thanks for the work, seems nice to have access so easily to my BeeWi sensor from HA ! 👍