mduran80 / daikin_madoka

Home Assistant custom component integration for the BRC1H thermostat (madoka)
MIT License
42 stars 10 forks source link

Futurewarning #18

Closed BeFygo closed 1 year ago

BeFygo commented 1 year ago

Hi,

The daikin_madoka integration work fine with HA 2022.8.2 but trows the error noted below:

connected = await self.client.is_connected() /usr/local/lib/python3.10/site-packages/pymadoka/connection.py:163: FutureWarning: is_connected has been changed to a property. Calling it as an async method will be removed in a future version DISCOVERED_DEVICES_CACHE = await scanner.get_discovered_devices() /usr/local/lib/python3.10/site-packages/pymadoka/connection.py:56: FutureWarning: This method will be removed in a future version of Bleak. Use the discovered_devices property instead.

Upgrading to HA 2022.9.4 the integration doesn't connect to the device anymore.

Maybe repairing fixes this, but I did not have the time yet to try.

Update: Even repairing the device doesn't work

mduran80 commented 1 year ago

I will have a look at the version uograde when I find some time (soon, hopefully).

The FutureWarnings have already been adressed in the pymadoka library, although I haven’t released the version yet until I can do more testing.

Since HA started using bleak too, more compatibility issues may arise and affect pymadoka stability.

El El mié, 14 sept 2022 a las 22:10, BeFygo @.***> escribió:

Hi,

The daikin_madoka integration work fine with HA 2022.8.2 but trows the rror noted below:

connected = await self.client.is_connected() /usr/local/lib/python3.10/site-packages/pymadoka/connection.py:163: FutureWarning: is_connected has been changed to a property. Calling it as an async method will be removed in a future version DISCOVERED_DEVICES_CACHE = await scanner.get_discovered_devices() /usr/local/lib/python3.10/site-packages/pymadoka/connection.py:56: FutureWarning: This method will be removed in a future version of Bleak. Use the discovered_devices property instead.

Upgrading to HA 2022.9.4 the integration doesn't connect to the device anymore.

Maybe repairing fixes this, but I did not have the time yet to try.

— Reply to this email directly, view it on GitHub https://github.com/mduran80/daikin_madoka/issues/18, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFK3CT5HFTJSPKH4B4C3ZBTV6IWLNANCNFSM6AAAAAAQMX6YLE . You are receiving this because you are subscribed to this thread.Message ID: @.***>

mduran80 commented 1 year ago

I just updated my HA and base OS and the integration is working as expected. However, take into account that HA updates may require to repeat all the bluetoothctl process so the info in the D-BUS is updated.

Tested with:

Home Assistant 2022.9.6 Supervisor 2022.09.1 Operating System 8.5 Frontend 20220907.2 - latest

Please connect via shell and remove the device:

  1. Run bluetoothctl
  2. Type remove
  3. If the integration is running in HA, the bluetootctl client should immediately show the connection confirmation request with the pair code, just accept it and rush to the thermostat to accept it too.
  4. The integration should work and show as available
BeFygo commented 1 year ago

I just upgraded to the latest HA 2022.9.6 and followed your advise, at first glance... the same problem.

After experimenting i found out that I had to activate the "bluetooth" integration to make madoka work with HA 2022.9.6. I disabled that integration in the past because it conflicted with madoka. Probably because i use the "unsupported" tp-link UB500.

So in a nutshell, it works (for now) and I'm going to order an other adapter hoping for the best.

Thank you for your help!!

mduran80 commented 1 year ago

I am using a TP-link UB400 myself. The conflicts might be caused because HA started to use bleak for the BT and its version was higher than pymadoka’s but should be fine now.

El El lun, 26 sept 2022 a las 14:17, BeFygo @.***> escribió:

I just upgraded to the latest HA 2022.9.6 and followed your advise, at first glance... the same problem.

After experimenting i found out that I had to activate the "bluetooth" integration to make madoka work with HA 2022.9.6. I disabled that integration in the past because it conflicted with madoka. Probably because i use the "unsupported" tp-link UB500.

So in a nutshell, it works (for now) and I'm going to order an other adapter hoping for the best.

Thank you for your help!!

— Reply to this email directly, view it on GitHub https://github.com/mduran80/daikin_madoka/issues/18#issuecomment-1257943904, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFK3CT3VRUMHMVFAIXOV5FTWAGH33ANCNFSM6AAAAAAQMX6YLE . You are receiving this because you commented.Message ID: @.***>

BeFygo commented 1 year ago

Oke, I'll wait with ordering a new adapter and keep you posted.

I am using a TP-link UB400 myself. The conflicts might be caused because HA started to use bleak for the BT and its version was higher than pymadoka’s but should be fine now. El El lun, 26 sept 2022 a las 14:17, BeFygo @.> escribió: I just upgraded to the latest HA 2022.9.6 and followed your advise, at first glance... the same problem. After experimenting i found out that I had to activate the "bluetooth" integration to make madoka work with HA 2022.9.6. I disabled that integration in the past because it conflicted with madoka. Probably because i use the "unsupported" tp-link UB500. So in a nutshell, it works (for now) and I'm going to order an other adapter hoping for the best. Thank you for your help!! — Reply to this email directly, view it on GitHub <#18 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFK3CT3VRUMHMVFAIXOV5FTWAGH33ANCNFSM6AAAAAAQMX6YLE . You are receiving this because you commented.Message ID: @.>

BeFygo commented 1 year ago

Hi,

Yesterday i noticed that the AC became "unavailable" several times en sometimes it stayed "available" but didn't represent the real status. Meaning that it would say its "off" while in realtime it was in "heat" mode (activated by the madoka thermostat).

So with my setup and HA 2022.9.6 madoka is unstable.

Setup: OMV 6 Version: 6.0.41-1 (Shaitan) Kernel: Linux 5.18.0-0.deb11.4-amd64 HA: Docker Bluetooth adapter: TP-link UB500