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

Add support for water heater devices and the Hydrotherm DYNAMIC/X8 heat pump hot water system #251

Merged
merged 2 commits into from
Oct 26, 2022

Conversation

fsevilla3
Copy link
Contributor

Hi @make-all. Thanks for your work on tuya-local. I love how clean and abstracted your code is. Please accept this pull request, which adds generic support for water_heater devices and support for the Hydrotherm DYNAMIC/X8 heat pump hot water system. I've tested this and it works for me.

@make-all make-all merged commit e5b8e19 into make-all:main Oct 26, 2022
@make-all
Copy link
Owner

Thank you, I had thought about water_heater support for the pool heaters that are supported but hadn't yet gotten around to figuring out the differences compared to climate.

make-all added a commit that referenced this pull request Oct 26, 2022
- Tests for PR #251
- Allow setting temperature in the general case, make it readonly for Hydrotherm device in the config.
- Add error_code as attribute for Hydrotherm device so more detailed error info can be read.
- remove mapping to scale by 1 (not required) and additional "off" values.
@Spalfnaganalf1
Copy link

Out of curiosity - does your Hydrotherm still show 50 degrees when its actually reached the set temp of 60?

@fsevilla3
Copy link
Contributor Author

Out of curiosity - does your Hydrotherm still show 50 degrees when its actually reached the set temp of 60?

Yes, my reading also stop at 50C when actual is at or above 50C. This is the same behaviour with the Tuya mobile app, so it’s not an issue with the HACS integration. I’ve reported it to the Hydrotherm team but I don’t know if it’s something they can or will fix (eg via OTA update).

@Spalfnaganalf1
Copy link

Yes, my reading also stop at 50C when actual is at or above 50C. This is the same behaviour with the Tuya mobile app, so it’s not an issue with the HACS integration. I’ve reported it to the Hydrotherm team but I don’t know if it’s something they can or will fix (eg via OTA update).

No worries - For now i've just re-entered the 'scale' part of the code that was removed and i am using 0.8333333333333 as the scale. Not sure if it is accurate for anything outside of the 60 degrees, but at least i'm not being triggered seeing 50 degrees anymore.

@curik8
Copy link

curik8 commented Apr 30, 2023

Hi! Having the same issue here regarding max 50 deg. Is there a solution for this yet?

@make-all
Copy link
Owner

You would have to ask Hydrotherm. There is nothing that a Home Assistant integration can do about faulty devices.

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

Successfully merging this pull request may close these issues.

None yet

4 participants