-
Notifications
You must be signed in to change notification settings - Fork 43
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
General Discussions #70
Comments
I'm still getting over the excitement of R1.9. Good job! |
It is pretty damm complete now.. I think we should focus on getting into HA as a native component... I was reviewing docs yesterday and I think its pretty much done.. Give it a couple of weeks so we can make sure there isnt anything we should fix/enhance at the same time then we'll look at it.. |
Hey all, Mark has been super busy and made some changes which add config flow to the integration. Im not sure what else we need to do on this before publishing it to HA as a native component? Can people please test dev? |
HA version 0.107.1 Dev branch testing. Getting this error at one minute intervals. 2020-03-20 12:49:10 ERROR (MainThread) [homeassistant.core] Error doing job: Task exception was never retrieved Not sure if it's related but I don't think I had these water heater services before the update. |
I'll investigate tonight.. I didnt see them in my build, i wonder why... , @msp1974 fyi |
Hi, sorry been quiet for a while. Been dealing with the new C word! This is my fault, I was playing with adding a water heater component to manage the hot water functions. I wont load if you do not have hot water capability on your hub and I dont have that on mine (only single channel). I should have pointed it out more clearly and that it needed testing by someone who has. Couple of options
Happy to do which ever option as my fault. |
I do actually have a hot water channel on my heat hub and use it daily via a schedule. I'm happy to test and report back if you have any changes to suggest that I could make. |
I'm puzzled, this is water_heater is for the hot water system right? I've looked at my wiser app and I don't have the option to set the temperature or the "mode". Currently we already have set_water_mode (on/off/auto) and I think the only one missing is the ability to set the away mode affects hw Perhaps just back these out until they are better understood? |
I think that is probably right. i was looking at it as I had seen a few smart thermostats use it to manage HW. I think it is a good solution to manage the hot water functions as if you add the entity for this, you can change the mode from the UI and it has built in services, but probably best as a branch until fully understood and tested. Will send you a PR to back out (small change to init and remove water_heater.py) and then create another branch for this. |
ok, I think thats best.. Im not convinced its needed...., look forward to the PR.. Thanks |
@Fulch , all checked in and appears to be ok now , with no errors. Can we check this, if its all ok, I'll make a 2.0 release then 100% focus on pushing to native HA I mean with this corona stuff theres nothing else to do eh? |
Running the latest Dev release since late yesterday and everything looks good! |
Same here.. I'll give it to the weekend and then push it as a 2.0 release.. |
So far it works fine so I will release 2.0 this weekend, just saw a issue (#79) which was closed by toby (unprintable characters in the SSID screwing up JSON).. Its not "really" a bug, the JSON produced is invalid. Toby may produce a PR and if he does it this weekend I'll wait for it. not critical TBH~ Surprisingly cold outside isnt it |
Hi all, I have the water heater functionality on my Wiser system. I think the reason for having the water heater component is so that you can use the Boost 30m/1h/2h/3h preset functionality, in the same way you can for each room. I'm happy to do some testing if needed. |
Hey all, anyone else got the same issue as m01 on entities (#81)? @TobyLL , yeah makes sense, will do for 2.1 |
Published 2.0 :-) |
So how is all doing with self-isolation.. For me its turned out to mean I have even less time! (childcare etc) |
moved to my parents place, away from the city to a more isolated landside spot. equipped them with a rpi hassos a couple of weeks ago and now setting up some basic automations and watch the corona sensor change states. hope yall good! |
it appears that 2.0 release is broken |
Logged #83 |
Using 2.0 release. 2020-03-30 00:05:52 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry WiserHeat036CB1 for wiser |
created patch, see #83 , available for testing. (reason for bug is wiser upgraded the hub firmware and broke us) |
Ive pulled the R2.0 Release from HACS, until we are sure whats broken it its best to not use it.. Love firmware updates |
@scooper1 fyi
Have posted this on another issue also but don't know how to link that comment into this issue so apologies for posting it twice. the config flow in v2.0 of this component will do one of 3 things.
|
still broken for me Traceback (most recent call last): 2020-03-31 00:25:55 ERROR (MainThread) [aiohttp.server] Error handling request |
logged #110 to remind me |
guys @msp1974 looks like I need to log a number of PRs to get this integration included into HA core There will be a period where HA Core will have the Wiser API but only a subset of it.. So.. How important is it to have this integration within HA Core? or is HACS just fine please vote by thumbs up 👍 if its important or thumbs down 👎 if HACS is just fine or don't care thanks |
mmmm 83% of people say "stay with HACS" Oh well thats it then! |
Loving this new way of being able to use my Wiser heating controls. I have quite a distributed network so have been experimenting with positions of smart plugs to improve signal coverage and originally the only way to get an update on device signal was by emailing Wiser support each time I moved a plug! Anyway; Is it possible to get access to the Opentherm data used by some of the Wiser base modules? My existing boiler does not have Opentherm, however if I could get a reading for the demand from the Wiser Opentherm module, I could use a servo to vary the circulation temperature knob on the boiler. |
Is the poll still open for integrating into HA? 🙂 It's not that surprising that users on here are ok with HACS, people that aren't might not even know your brilliant integration exists! I'd be really up for helping do the work to get it moved over if that's what's holding it back.. I can understand that there are lots of other reasons why you wouldn't want to do it though... |
@technofreak74 the reason we backed off from moving this into HA core was the fact that they wouldn't accept it as is. Ie we had done all the recoding to pass all the tests but we were asked to strip it back to a very low functional integration to be accepted. This would mean that everyone who is using today would loose a lot of the functionality until we could get enhancements reviewed and implemented in future HA releases. As this could be months, we decided that we did not want to put our user base through such a painful process just to get into HA core. |
As a idea could we split in two versions. One basic in core and advance versions in hacs is that allowed Has anyone talked to manufacture to get a bit of sponsorship for the core version On 29 Oct 2020 17:23, Mark Parker <notifications@github.com> wrote:
@technofreak74 the reason we backed off from moving this into HA core was the fact that they wouldn't accept it as is. Ie we had done all the recoding to pass all the tests but we were asked to strip it back to a very low functional integration to be accepted. This would mean that everyone who is using today would loose a lot of the functionality until we could get enhancements reviewed and implemented in future HA releases. As this could be months, we decided that we did not want to put our user base through such a painful process just to get into HA core.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.
|
That makes sense. It is a shame though, you guys have done so much work and it would definitely reach more people that way (I got HACS to use this integration!). You presumably lose a lot of control over the integration too if you did it. I think that is why they like all the heavy lifting to be done in a separate python lib, so they don't have to review it 😄. |
I think longer term it would be better for this integration to live in the Home Assistant core. In preparation for that, it might firstly make sense to check if there's any non-HomeAssistant specific code in this repo, and move it to the The second thing I would consider is figuring out the logistics of moving the code over in small chunks. Some ideas:
This repo doesn't feel soooo big that this problem could not be solved. I do also understand that as open source maintainers you may not deem this to be sufficiently important to warrant the extra work. In general I wouldn't mind helping with giving the move a go, but I'd have to check with my employer and find time as well. |
Hey @technofreak74 technofreak74 , I got annoyed with the process. This integration is quite advanced and complete and to publish it as a native HA integration we would need to break it down to a single platform and then publish it, then adding sensors/platforms one at a time until its all published. I explained to the HA guys that its kinda silly , we've used HACS as an incubator and now deciding to publish it on HA natively (for the reasons you mention) and they said tough... I'll be honest , the integration works, people know about it through the forums and thought.. meh.. not worth the effort.. |
ps baby now 6 weeks old, life getting back to normal :) and Yes @msp1974 been baby sitting for me :-) best partner I could have :-) |
@m01 totally agree.. I moved most code out to the wiser-heating-api module as you say, but you are right there might be one/two more bits which have organically added themselves... I'll check.
From what I understood, we can issue 10 Cumulative PRs and as you say hit merge as soon as the previous one is done.. I may revisit the idea of going to HA.... Sigh.. |
Just an update for all. I've been offline for a number of reasons recently, one of the reasons is we've moved house! Whoop whoop but also stressful. Now settled in and today Wiser is finally installed, however my HA machine isnt up and running yet, but will be soon. Mean whilst, sods law, HA changes caused our integration to break .. boo hoo, thankfully my co-collaborator and co-author of the integration (Mark Parker @msp1974) , has been fantastic in manning the fort and fixing a ton of changes today! Thanks Mark!!! Thanks all!! |
Are there any events generated when the climate.wiser changes temperature? I am trying to keep a (display only) Nest thermostat in sync with the (real) Wiser system. Using Nodered I can read the temperatures from each, and react if I find it has changed, but I'd rather have an event notify me, than a continuous loop. |
Currently there are no events in the integration. However, the integration only polls the hub for updates every x seconds, depending on your options settings. So you could reduce the overhead by only reading every x seconds too.We can look at this for future releases or happy for a PR if you want to do that.MarkSent from my Galaxy
-------- Original message --------From: olbasoil ***@***.***> Date: 02/11/2021 10:11 (GMT+00:00) To: asantaga/wiserHomeAssistantPlatform ***@***.***> Cc: Mark Parker ***@***.***>, Mention ***@***.***> Subject: Re: [asantaga/wiserHomeAssistantPlatform] General Discussions (#70)
Are there any events generated when the climate.wiser changes temperature? I am trying to keep a (display only) Nest thermostat in sync with the (real) Wiser system. Using Nodered I can read the temperatures from each, and react if I find it has changed, but I'd rather have an event notify me, than a continuous loop.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.
|
thats great news! and yes msp is doing a great job! with the recent changes to hassio's docker runtime I decided to seperate hassio completly from my docker stack to remain "supported" ....turned out to be much more work that I expected :) |
@olbasoil sorry its a little late but got chance this week to look at your request and i think you can already do in standard HA. I will put this in the recipees when i get chance for others if needed but below is an automation that will fire when target temp is changed and call a service to set target temp on another climate device. Not sure whether this is exactly what you wanted but maybe give you the idea to do your use case. If you want to use current temp instead of target temp use current_temperature as attribute. For the service, call whatever service/climate device is your nest thermostat.
|
Should also say that there is already a trigger on a climate device for current_temperature_changed but it seems to need to have a from and/or to value needed so i cant see how to make it trigger for any temp change. You could also look into that or I will look further when i get time. |
Really loving this integration - thank you so much! |
Didn't know about this, I'm eligible and as you say would be really cool to integrate - It may be agnostic of the existing kit though as the new device connects to WiFi rather than the ZigBee network 🤔 |
Looks interesting but i cant work out if Im eligible... |
@asantaga - had the same question. Figured out that after I updated to the latest app, there is a splash at the top of the Insights page that you can start a questionnaire. First question is if your address matches some database. I stopped there as (a) I dont have a smart meter and (b) I'm a little neurotic and I gave a false address! |
I completed the eligibility check, but was told I was not eligible because
my smart meter was not SMETS2. It is SMETS1 but there has been a program to
upgrade SMETS1 to be SMETS2 compatible (mostly to allow a free market in
changing energy supplier) and mine is registered in the national database,
so there is probably some more obscure criteria. I was on a support call to
Drayton yesterday about something else, and mentioned this. and hope they
will come back with an answer. It is impossible to get a working SMETS1
replaced with a SMRTS2 (though non smart meters can be replaced).
…On Wed, Nov 24, 2021 at 2:43 AM wizmo2 ***@***.***> wrote:
Looks interesting but i cant work out if Im eligible...
@asantaga <https://github.com/asantaga> - had the same question. Figured
out that after I updated to the latest app, there is a splash at the top of
the Insights page that you can start a questionnaire. First question is if
your address matches some database.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#70 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKVCV33STFYRC24QTWABHXTUNRGMBANCNFSM4K2GQLYQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
An update: The monitor arrived today - it's a Geo Trio II. I connected it to my wifi and after a few hours it had magically linked to my smart meter (I guess when I gave the number as the eligibility check they stored it to link against). |
Im in same boat.. Waiting for my no so smart meter to die :-) |
NIce... Im feeling left out (see comment above) |
Im sure everyone knows this but there is also a healthy discussion on https://community.home-assistant.io/t/drayton-wiser-home-assistant-integration/80965/292eibcccvntcdlbkrnvidlrH |
Im sure everyone knows this but there is also a healthy discussion on https://community.home-assistant.io/t/drayton-wiser-home-assistant-integration |
Hey all, feel free to use this issue to ask questions ideas for our next release 2.0./
For bugs please log an issue
For other queries either use this issue or the Home Assistant forum https://community.home-assistant.io/t/drayton-wiser-home-assistant-integration/80965/47
thanks!
The text was updated successfully, but these errors were encountered: