itchannel / apex-ha

Local Neptune Apex HA Integration (Aquarium Controller)
GNU General Public License v3.0
22 stars 3 forks source link

FEATURE - Poll frequency #3

Closed behold81 closed 2 years ago

behold81 commented 2 years ago

Firstly, I was just trying to work out this and you have just done this for me. you are awesome.

It would be great if you could control the poll interval if possible. some might want it only every x mins to remove load from the poll. I assume switches are not an issue as it calls an API to enact a switch.

Maybe its not suitible but I thought I would float the idea.

itchannel commented 2 years ago

Good idea now available in the latest release 1.05.

Update interval (In Minutes) can be set in options

behold81 commented 2 years ago

image

There is not description on the configure button for the option. I assume it is on the initial configure but not this screen.

itchannel commented 2 years ago

It is on mine. Try doing CTRL R in your browser as probably a cache issue.

image

behold81 commented 2 years ago

It finally caught up. I thought I had done that but apparently not. Ill go sit back in my box!!

mlmhhomeauto commented 2 years ago

It is great to have the option of the poll interval, but for my usage it would be an improvement to have the option to be in seconds vs minutes. Using the smaller measurement still allows for minutes as in setting the poll interval to 300 sec would equal 5 minutes.

behold81 commented 2 years ago

Its a fair point but be aware polling too frequently will impact the apex brain and cause it to slow and not work.

itchannel commented 2 years ago

@mlmhhomeauto I did start with seconds when I was programming it but my Apex was very unstable polling quicker than a minute. The newer version might be more stable but the older models just aren't very powerful from my findings.

Happy to change it to seconds if people would prefer it just put minutes to avoid people complaining when it doesn't poll every 5 seconds 👍

behold81 commented 2 years ago

Will it accept 0.25 mins? (So 15 seconds?) although many programming systems show in second or milliseconds even for minute use.

Maybe set at 60s default and put a warning on it.

itchannel commented 2 years ago

Will change it to seconds in the next release. Will have to be a disclaimer if anyone has it set in minutes though on this release 👍