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.

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

Wiser rooms do not stay set to off #95

Closed
msp1974 opened this issue Apr 15, 2020 · 20 comments
Closed

Wiser rooms do not stay set to off #95

msp1974 opened this issue Apr 15, 2020 · 20 comments
Assignees
Milestone

Comments

@msp1974
Copy link
Collaborator

msp1974 commented Apr 15, 2020

Don't think this is an integration issue but wondering if anyone else has seen this issue.

As the warmer weather has come, I have set some of my rooms to off (as I haven't got around to creating a summer schedule yet). However, (and I thought I was going mad at first) I keep noticing that the next day, some of the rooms are set to manual heat with a temp higher than the away set temp but not all.

It looks (from yesterday) that this happened at 11:26. Has anyone else seen this behaviour with their hub?

@scooper1
Copy link

scooper1 commented Apr 15, 2020 via email

@asantaga
Copy link
Owner

asantaga commented Apr 15, 2020

im not seeing this.. I have a front room which is set to off and its still off..

I did notice something similar to this when the battery was low.. but more like the rad was heating and reporting back the temp..

edit : That said Ive set my to off from the official wiser app not the integration.. I think thats the difference..

I think when we're setting it to off its turning out to be a override "off" vs setting the mode to off.. Will look at the rest api and see if we can make this change..

@msp1974
Copy link
Collaborator Author

msp1974 commented Apr 15, 2020

I'm seeing both somehow. I have some that just show off and some that show off- override. Was going to have a look at this too to see if we should cancel any override first. Have got 5 showing off and 3 showing off - override. Will see what happens to these later.

@asantaga
Copy link
Owner

image
dummy is a dummy room which I use for testing...

@msp1974
Copy link
Collaborator Author

msp1974 commented Apr 15, 2020

Finding if you set of off, it is now more often than not going to off - override. However, if you go to manual mode, change temp and then set to off, it is always off (if you don't change temp then goes to override too). Wonder if we should do a 2 step process to turn off. 1: to set to manual mode 2: set temp to -200.

Having said this, still haven't worked out if it is just these ones that say off - override that are going back to manual mode or a random selection. Will report back over next 24 hours if I can work it out.

@scooper1
Copy link

scooper1 commented Apr 15, 2020 via email

@asantaga
Copy link
Owner

actually yeah, both of the rads I set to off using HA are now back on , so yeah the turning off isnt working :-) and :-(

We'll work it out... we always do!

@djashjones
Copy link

djashjones commented Apr 17, 2020

I've noticed that one of my TRV's for the past couple of days comes on when the heating does and it's schedule is pretty much off i.e. set to 14 degrees but my other TRVs are fine for some reason.

Scrap that: My fault. Think it came loose. Re-calibrated and it's fine. Sorry!

@m01
Copy link

m01 commented Apr 26, 2020

I've noticed this issue as well, but generally it only happens once, i.e. the turning off works the 2nd time.
Just a totally random guess, but I wonder if it's something frost-protection related. I've noticed that if you set the room stat to 5°C it has the same white circle as off, but I vaguely remember the minimum temperature in the home-assistant integration being 15 or so...
I don't currently have any TRVs in case it makes a difference.

@msp1974
Copy link
Collaborator Author

msp1974 commented Apr 30, 2020

PR for the api posted that I think fixes this

@asantaga
Copy link
Owner

asantaga commented May 3, 2020

Needs testing.. will add to readme

@asantaga asantaga assigned asantaga and msp1974 and unassigned asantaga May 3, 2020
@asantaga asantaga added this to the 2.3 milestone May 3, 2020
@msp1974
Copy link
Collaborator Author

msp1974 commented May 3, 2020

@asantaga just to make sure you haven't missed the PR for the wiser-heating-api that needs to be merged to fix this issue. Best to do with 2.3 release I think.

@asantaga
Copy link
Owner

asantaga commented May 6, 2020

Hey mark, I think this PR is in 2.3.1 right?

@msp1974
Copy link
Collaborator Author

msp1974 commented May 6, 2020

No. The fix for this is actually an update to the api not the HA integration. The PR is still open for the api.

@asantaga
Copy link
Owner

PR Merged into a new wisehubapi, 1.0.8 ..

I need to think how to document how to force an API upgrade as I dont think HA automatically does it on boot

@msp1974
Copy link
Collaborator Author

msp1974 commented May 13, 2020

don't you just update the version in manifest.json and issue a new release?

@asantaga
Copy link
Owner

asantaga commented May 13, 2020

yup, but i hate to create new releases just to force a library update.. although thats what Im going to do later tonight..

you can also upgrade the library by issueing the pip3 --upgrade command.. from the virtual env or within the docker container.

but it would be nice if HA detected the later version and just upgraded it, especially as we've said min version not exact version

@msp1974
Copy link
Collaborator Author

msp1974 commented May 13, 2020

True but ive seen lots of integrations do a version update just to upgrade the library its using so wouldnt worry.

@asantaga
Copy link
Owner

Will release this with 0.110 compatible release

@asantaga
Copy link
Owner

fixed, and tested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants