Trouble with the new boost function about wiserhomeassistantplatform HOT 15 CLOSED

Forgot to add, I've got 1.7.1 running on a Pi3 with Hassio 103.4.

from wiserhomeassistantplatform.

mmmm something it doesnt like at the REST API place.. im sure i tested this too...

Alas my server (my whole HA installation ) is down and I dont have the time to rebuild it yet (got to do a complex tax return first....)

from wiserhomeassistantplatform.

@msp1974 Hey Mark, given my server is down and time poor, any chance you have time to look at this?
Happy to accept any PR and merge as always

from wiserhomeassistantplatform.

server back up :-) will work on this asap

from wiserhomeassistantplatform.

Awesome, thank you!

from wiserhomeassistantplatform.

Sorry for late response i have been away. I can look at the weekend if you need some help. I found that the error message in the api could do with enhancimg to log the return response better so we can diagnose issues more easily.

from wiserhomeassistantplatform.

The one thing i can think off that may cause an error that is not being checked is your boost_temp setting. The boost function takes the current temp of the room and adds the boost_temp value to it. This was a breaking change from prior version.

If you still have boost_temp set to something like 20 and not something like 2, current temp + boost temp will be more than 30 which is the max temp setting of the wiser system and it will error when setting boost mode.

There is no checking for the min/max temp range when setting the boost in the api like we have when setting a temp value. Maybe we need to add this and return a message to explain the issue.

from wiserhomeassistantplatform.

Other possibility is it is being caused by the open issue 3 on the api that it does not always send an integer to wiser but sends a float value.

from wiserhomeassistantplatform.

The one thing i can think off that may cause an error that is not being checked is your boost_temp setting. The boost function takes the current temp of the
If you still have boost_temp set to something like 20 and not something like 2, current temp + boost temp will be more than 30 which is the max temp setting of the wiser system and it will error when setting boost mode.

Thanks Mark, where would I find boost_temp? I definitely haven't deliberately changed anything from the defaults.

from wiserhomeassistantplatform.

It will be in your configuration.yaml file where you have your settings for IP, secretkey etc.

from wiserhomeassistantplatform.

I'm a muppet, boost temp was indeed set to 20. Sorry for the trouble and thank you both for your help.

from wiserhomeassistantplatform.

ReOpening this, Server is back but work and kids are also demanding. I plan to make some changes this weekend , merge a PR and what I'll do is add some checks for boost values > 10..

from wiserhomeassistantplatform.

Have sent you a PR on the wiserHubAPI to fix this.

KR

from wiserhomeassistantplatform.

Thanks.. really appreciate this

Ill.merge into master tonight/ tomorrow morning and make a new HCAS release.

from wiserhomeassistantplatform.

thank you, closing this and creating a new HACS release

from wiserhomeassistantplatform.