iMicknl / ha-tahoma

Custom component for Home Assistant to interact with smart devices via Somfy TaHoma or other OverKiz based API's.
MIT License
151 stars 30 forks source link

use of deprecated 'async_get_registry', use async_get instead... #809

Closed browetd closed 1 year ago

browetd commented 2 years ago

Did you read the instructions?

The problem

At startup and since the last release of HA (2022.6), I have the following errors while ha-tahoma is starting:

2022-06-05 09:27:15 WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that uses deprecatedasync_get_registryto access device registry, use async_get instead. Please report issue to the custom component author for tahoma using this method at custom_components/tahoma/__init__.py, line 141: device_registry = await dr.async_get_registry(hass) 2022-06-05 09:27:15 WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that uses deprecatedasync_get_registryto access entity registry, use async_get instead. Please report issue to the custom component author for tahoma using this method at custom_components/tahoma/climate_devices/atlantic_electrical_heater_with_adjustable_temperature_setpoint.py, line 109: entity_registry = await self.hass.helpers.entity_registry.async_get_registry()

What version of this integration (ha-tahoma) has the issue?

V2.13.0

What version of Home Assistant Core has the issue?

2022.6.2

Which gateway / hub do you use?

Cozytouch

Device model

Equateur 3

Home Assistant log

Logs ``` Copy/paste any log here, between the starting and ending backticks. The first log line must start with "Added device (Device...". ```

See problem description...

Additional information

No response

tetienne commented 2 years ago

Hi, thx for reporting this. Depending of the HA reviewer, your device will be soon available directly from the Overkiz integration and so you will no need to use this custom component: https://github.com/home-assistant/core/pull/72790

github-actions[bot] commented 2 years ago

'There hasn't been any activity on this issue recently. Is this issue still present? Please make sure to update to the latest Home Assistant version and version of this integration to see if that solves the issue. Let us know if that works for you by adding a comment πŸ‘. This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.'

roofburner commented 2 years ago

Still present: 2022-07-06 12:50:29 WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that uses deprecated async_get_registry to access device registry, use async_get instead. Please report issue to the custom component author for tahoma using this method at custom_components/tahoma/init.py, line 141: device_registry = await dr.async_get_registry(hass)

browetd commented 2 years ago

@tetienne I am using Overkiz Integration (latest release) and the issue is still present. There is two references to async_get_registry in init.py at lines 141 and 158 of the current code.

tetienne commented 2 years ago

@browetd what is the device you need we implement within Core to leave this custom component?

browetd commented 2 years ago

@tetienne I am using three "Equateur 3" heaters from Atlantic via cozytouch... Is it already supported by Core ? I have seen that a PR has been issued recently but have not seen any mention of that in the last release of HA 2022.7.0 (I am still using 2022.6.7 due to the amount of potential problems generated by the move to Python 3.10)...

github-actions[bot] commented 1 year ago

'There hasn't been any activity on this issue recently. Is this issue still present? Please make sure to update to the latest Home Assistant version and version of this integration to see if that solves the issue. Let us know if that works for you by adding a comment πŸ‘. This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.'

browetd commented 1 year ago

Is the Equateur 3 devices (Brand: Thermor using Cozytouch) are supported by the integration in core Home assistant ? If not, I still have the issue...

iMicknl commented 1 year ago

@browetd this still depends on https://github.com/home-assistant/core/pull/72790. Which is not merged yet.

github-actions[bot] commented 1 year ago

'There hasn't been any activity on this issue recently. Is this issue still present? Please make sure to update to the latest Home Assistant version and version of this integration to see if that solves the issue. Let us know if that works for you by adding a comment πŸ‘. This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.'

roofburner commented 1 year ago

Still present: 2022-09-28 22:20:53.763 WARNING (MainThread) [homeassistant.helpers.frame] Detected integration that uses deprecated async_get_registry to access device registry, use async_get instead. Please report issue to the custom integration author for tahoma using this method at custom_components/tahoma/init.py, line 141: device_registry = await dr.async_get_registry(hass)

iMicknl commented 1 year ago

Does this block the integration / home assistant from functioning? We are aware of this issue and hope to have the components migrated as soon as possible.

browetd commented 1 year ago

@iMicknl As far as I know, this is not blocking the integration to work... Just warning messages... Thank you for your hard work on this integration. I appreciate

roofburner commented 1 year ago

All works well, for now...Op 7 okt. 2022 18:21 schreef Mick Vleeshouwer @.***>: Does this block the integration / home assistant from functioning? We are aware of this issue and hope to have the components migrated as soon as possible.

β€”Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

iMicknl commented 1 year ago

Fixed in latest version. Thanks for your patience @browetd ;-).