search

mchwalisz / home-assistant-senec

SENEC Battery integration for Home Assistant
Apache License 2.0
46 stars 23 forks source link

Make scan interval configurable #29

Open LavermanJJ opened 2 years ago

LavermanJJ commented 2 years ago

Hi - thanks for providing this integration! I wanted to ask, if it would be possible to make the ’SCAN_INTERVAL‘ configurable?

Or if there is a specific reason why it isn't? I know, that there is a minimum delay between two polls, but 10s seems to be still working for me.

Thanks, Jeroen

guevara777 commented 2 years ago

I would love to see an option to manually set the scan_interval.

mchwalisz commented 2 years ago

Scan interval should be controlled by home assistant. At least, that's I intended to implement it based on HA docs (cannot find details now).

The biggest roadblock is implementing the config flow.

LavermanJJ commented 2 years ago

That matches my understanding. Unfortunately, I've no experience in developing home-assistant integrations. However, I've tried to look around how other integrations do it. The examples, I've found seem to do it in a similar fashion. Maybe this helps?

RikyUnreal commented 1 year ago

Hi all, this great integration work smoothly with my SENEC.Home V3 Hybrid, but, me too, I would like the system to be polled a little more frequently. 10 seconds as suggested by @LavermanJJ will be great!

lennartschulte commented 1 year ago

I don't see the "Configure" Option in the new HA integration overview as shown in the pull request.

marq24 commented 1 year ago

since the PR is not merged yet