dlarrick / hass-kumo

Home Assistant module interfacing with Mitsubishi mini-split units
MIT License
96 stars 21 forks source link

Timeout Values not using DataUpdateCoordinator #66

Closed omriasta closed 1 year ago

omriasta commented 2 years ago

Wouldn't it be beneficial to let homeassistant set the times to update the entities using the DataUpdateCoordinator: https://developers.home-assistant.io/docs/integration_fetching_data/

I don't see that implemented and wondering if this would also remove a lot of the log messages that we get when there is a timeout..maybe the 5 second update is a bit much for most entities?

danielgoepp commented 2 years ago

I haven't looked into this in detail, but at first glance seems like a good idea :) +1

dlarrick commented 1 year ago

Closing some old tickets. Kumo uses DataUpdateCoordinator as of the recent re-org.