sHedC / homeassistant-mastertherm

Home Assistant Mastertherm Component, to communicate and control heat pumps from Mastertherm
MIT License
6 stars 2 forks source link

Add Periodic Full Data Reload and Refresh #93

Closed sHedC closed 1 year ago

sHedC commented 1 year ago

Checklist

Is your feature request related to a problem? Please describe.

It has been seen a couple of times, if the HP does not respond correctly with updated data the integration can miss the update until the token times out and a full data re-fresh is done.

Describe the solution you'd like

Have a periodic full data load and sensor refresh to minimize the impact of this happening. Also maybe a full data load if any warning or temporary error happens.

Also check if maybe the last refresh date can be altered to retrieve updates from a previous time rather than relying on the update date the Mastertherm Web API gives.

Describe alternatives you've considered

N/A

Additional context

One scenario seen is if a set it done the HP does not respond with the update on the next load, it is possible this is related to timezone issues.

sHedC commented 1 year ago

@johny-mnemonic , @valentapat

Added an offset of default: -5 seconds in the incremental updates so it takes the last update time/date and takes off 5 seconds before making a new call.

Also added a full data refresh of default 15 min

Both can be configured in the Options.