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

Support Domestic Hot Water #143

Merged
merged 14 commits into from
Mar 9, 2023
Merged

Support Domestic Hot Water #143

merged 14 commits into from
Mar 9, 2023

Conversation

georgesdib
Copy link

@georgesdib georgesdib commented Feb 18, 2023

Added 2 HomeKit accessories, a temperature sensor and a switch to support Evohome Domestic Hot Water.

Also added history logging for Eve.

Changed the behaviour of the valve for the thermostat to be 100% when it's heating and 0% when it's not.

Fixes #80

@SeaneyC
Copy link

SeaneyC commented Feb 18, 2023 via email

@5xPa
Copy link

5xPa commented Feb 18, 2023

Have applied the update, I don't see anything to do with the Hotwater system. Is there something I need to enable?

@georgesdib
Copy link
Author

Have applied the update, I don't see anything to do with the Hotwater system. Is there something I need to enable?

After I updated the code, I just went to my home app and I could see the hot water temperature sensor and switch. Are you not able to see that at all? Not to state the obvious, but you are using it on a location that has hot water?

@5xPa
Copy link

5xPa commented Feb 18, 2023 via email

@SeaneyC
Copy link

SeaneyC commented Feb 18, 2023 via email

@georgesdib
Copy link
Author

Just so i’m clear, this hasn’t been merged into main yet has it? So the update won’t have made it to Homebridge yet, if that’s what you’ve updated?

That is correct, it has not been merged to main yet.
@5xPa are you using the official plugin from homebridge? If so, then indeed, it won't be there yet. You need to use this branch's code to have it.

@5xPa
Copy link

5xPa commented Feb 18, 2023 via email

@5xPa
Copy link

5xPa commented Feb 18, 2023 via email

@georgesdib
Copy link
Author

How do I i?nstall a branch

I would strongly advise running a second homebridge for that to not mess up with your production setup. If you don't feel comfortable for that, I would say don't test it yet.

Otherwise, if you have a separate homebridge running, you need to find where the plugins are installed, and override the homebridge-evohome files by the ones here and then restart homebridge (or only the sub bridge if you are running a sub bridge).

@5xPa
Copy link

5xPa commented Feb 18, 2023 via email

@luc-ass
Copy link
Owner

luc-ass commented Feb 20, 2023

Hey there. This is awesome news!! As I do not have the hot water unit I have not way of testing. How long have you been running it? Is it stable?

@georgesdib
Copy link
Author

Hey there. This is awesome news!! As I do not have the hot water unit I have not way of testing. How long have you been running it? Is it stable?

I have been running it for a couple of weeks now, seems pretty stable to me yes.
Please note I also changed the way the valve for a regular thermostat reports (instead of hardcoding to 50%, make it 100 or 0 based on if targetTemp >= currentTemp

@luc-ass
Copy link
Owner

luc-ass commented Feb 20, 2023

Great! I'll review the changes in the coming days and merge it back into the project. Thank you so much for your hard work!

@luc-ass
Copy link
Owner

luc-ass commented Mar 1, 2023

LGTM, will test it over the weekend

@luc-ass luc-ass merged commit 0f32f06 into luc-ass:master Mar 9, 2023
Copy link
Owner

@luc-ass luc-ass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@luc-ass
Copy link
Owner

luc-ass commented Mar 9, 2023

@georgesdib I've released your additions and fixes as v0.11.0. Thank you!! 🚀

@5xPa
Copy link

5xPa commented Mar 9, 2023

Brilliant can now view Hotwater in HomeKit. Thanks for all the effort

@georgesdib georgesdib deleted the valve-history branch March 9, 2023 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stored hot water support
4 participants