rospogrigio / daikin_residential

Cloud control of Daikin devices that are supported by Daikin Residential Controller app.
GNU General Public License v3.0
281 stars 37 forks source link

Reducing polling time to every 15 mins, due to incoming rate limitation #183

Open rospogrigio opened 6 months ago

rospogrigio commented 6 months ago

As suggested by @Daikin-Europe in #182 , the polling interval is now changed from 15 seconds to 15 minutes, since the calls rate will be limited to approx. 150 per day.

jwillemsen commented 6 months ago

Let us not merge this until there is a real rate limit in place, hopefully they will not add a rate limit at all

rospogrigio commented 6 months ago

Yes, I was thinking the same. Let's keep this PR ready in case we really need it.

rgerhards commented 6 months ago

based on #184, it looks like Daikin has already put in place some rate-limiting.

As a suggestion, I think it would be good to have a property for the polling period. That way user's can experimentally see how far the come. Right now, I have patched it to 90 secs in the hope it will help with #184.

barryvdh commented 6 months ago

Yes that would be nice. For automations it might be more important to be able to call the API reliably and use the HomeAssistant state as source, and update less often (or use an external temperature sensor)