Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make scan interval configurable #29

Open
LavermanJJ opened this issue May 24, 2022 · 6 comments · May be fixed by #41
Open

Make scan interval configurable #29

LavermanJJ opened this issue May 24, 2022 · 6 comments · May be fixed by #41

Comments

@LavermanJJ
Copy link

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
Copy link

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

@mchwalisz
Copy link
Owner

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
Copy link
Author

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
Copy link

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!

@LavermanJJ LavermanJJ linked a pull request Mar 3, 2023 that will close this issue
Open
@lennartschulte
Copy link

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

@marq24
Copy link

marq24 commented Jun 11, 2023

since the PR is not merged yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature configurable scan interval (#29) LavermanJJ/home-assistant-senec
6 participants
@mchwalisz @marq24 @LavermanJJ @guevara777 @RikyUnreal @lennartschulte