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

Aquatech Heat pump #1922

Closed
Mrsash2020 opened this issue May 16, 2024 · 7 comments
Closed

Aquatech Heat pump #1922

Mrsash2020 opened this issue May 16, 2024 · 7 comments

Comments

@Mrsash2020
Copy link

Describe the bug
Wanted to ask about another device but thought I can make it quick if ok. I am not sure how these things work so bear with me.

I have a new heat pump. I got it installed as it is tuya compatible. I have managed to add it in HA and using a tile card since don't have any options. Not sure if there are any other options

https://www.aquatechheatpumps.com.au/dynamic

The issue is the current water temp in the tank seems to indicate for example 17deg C which is not the current water temperature. Tuya shows me correct values. I have added screens of tuya app and also another screen in tuya which has other diagnostic temps etc.
Added HA screen showing the incorrect temperature as well.

So basically trying to figure out if the heat pump developer has stuffed up or is the data being read incorrectly by the integration. If the issue is the developer of the heat pump I can try and chase it up but the tuya app shows the info correctly, so was thinking something to do with integration and how it gets the data. I could be wrong...

I also note the tile card does not correctly show the heat pump modes. They should be Stand Water, Eco Water, Hyb Water, Hyb1 Water, ELE Water as per tuya but does have different names.

Expected behavior
Correct temp in the tank shown.

Additional context
Add any other context about the problem here.

If the bug involves a device, then please include device diagnostics from
config_entry-tuya_local-f5914d7e4059e1d693c31106ad430b9a.json

Please check if there are messages from Tuya Local in the Home
Assistant log (Settings / System / Logs), and include those too. None

If applicable, add screenshots to help explain your problem.
20240516_201805
Screenshot_20240516_201656_Tuya Smart
Screenshot_20240516_201726_Tuya Smart

@Mrsash2020 Mrsash2020 added the bug Something isn't working label May 16, 2024
@make-all
Copy link
Owner

This seems to be caused by PR #1686, which switched the displayed temperature from outlet to inlet temperature, because it matched what was displayed in their app, and they were getting garbage for the outlet temperature.

If the value in that case is always -50, then maybe it can be made conditional.

@make-all make-all added device improvement Improvement to an existing device config and removed bug Something isn't working labels May 16, 2024
make-all added a commit that referenced this issue May 16, 2024
An Aquatech Dynamic/X8 branded device appears to be compatible, but reports
the water temperature in dp 16, which was not present on the original
Hydrotherm branded device (it may in fact be gen 7). The inlet temperature
is reported as the cold water temperature at the inlet, as expected, and not
the hot water temperature as PR #1686 observed.  Both devices report outlet
temperature as -50, so that appears to be unused.

To handle these together, an optional dp 16 was added as the
current_temperature, which redirects to inlet_temperature if it is missing.

Issue #1922
@make-all make-all added awaiting confirmation Wating for confirmation the issue is solved unreleased Will be in next release labels May 16, 2024
@make-all
Copy link
Owner

Outlet temperature is reported as -50, same as the original report, but your device seems to be reporting an additional dp 16 with data matching the current temperature in your screenshots. I have added this as the primary current_temperature, with a redirect to inlet temperature when it is missing (as in the original report for this device).

The mode naming is based on predefined modes in Home Assistant. My preference is to fit in with HA rather than strictly following Tuya.

Stand (abbreviation for Standard) = Heat pump
HYB = High demand
HYB1 = Performance
ELE = Electric

The decision of which Hybrid option was which was based on the description in the manual, but this was for the Gen5 device.

@Mrsash2020
Copy link
Author

The modes I am ok with I guess as its not too much of an issue but the temperature I have no idea what you mean. So water temperature data is missing?

@matheuscanela
Copy link

I also notice the same issue on mine, I can see the bug fixed in the code, do you know when it is going to be merged?

@make-all
Copy link
Owner

It is merged already

@matheuscanela
Copy link

sorry, I meant released

@make-all make-all removed awaiting confirmation Wating for confirmation the issue is solved unreleased Will be in next release device improvement Improvement to an existing device config labels May 25, 2024
@Mrsash2020
Copy link
Author

It is merged already

I was wondering why mine wasn't showing. I had to again go to hacs for the update to show. Curious when was it released?

BTW thank you so much for fixing this.

timlaing pushed a commit to timlaing/tuya-local that referenced this issue Aug 8, 2024
An Aquatech Dynamic/X8 branded device appears to be compatible, but reports
the water temperature in dp 16, which was not present on the original
Hydrotherm branded device (it may in fact be gen 7). The inlet temperature
is reported as the cold water temperature at the inlet, as expected, and not
the hot water temperature as PR make-all#1686 observed.  Both devices report outlet
temperature as -50, so that appears to be unused.

To handle these together, an optional dp 16 was added as the
current_temperature, which redirects to inlet_temperature if it is missing.

Issue make-all#1922
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

3 participants