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.

Sign up for GitHub

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

Jump to bottom

Bug(CD): Heat Pump Water Heater RSJRAC01 wrong temperature #347

Closed
Punisher01 opened this issue Oct 7, 2024 · 31 comments · Fixed by midea-lan/midea-local#345
Closed

Bug(CD): Heat Pump Water Heater RSJRAC01 wrong temperature #347

Punisher01 opened this issue Oct 7, 2024 · 31 comments · Fixed by midea-lan/midea-local#345
Labels
bug Something isn't working
Milestone
0.6.6

Comments

@Punisher01
Copy link

HA core version

2024.10

Current integration version

v0.6.1

Last known working integration version

Device type and model

RSJRAC01

Working mobile app

SmartHome

The description of problem

The temperatures values are wrong: 10 degrees instead of 50 and also the current temperature is wrong. This problem persists from old unmaintained version.

Screenshot 2024-10-07 145324

The logs

config_entry-midea_ac_lan-30c54274be3917a9b39bb6660eb613b1.json
@Punisher01 Punisher01 added the bug Something isn't working label Oct 7, 2024
@chemelli74
Copy link
Collaborator

Hi @Punisher01, please enable debug for Midea AC LAN integration, wait for this error, disable debug and attach the log.

Note: it is better to drag the log into the comment (which will add it as an attachment) and not copy paste as it is hard to read logs in GitHub.

Thanks

@Punisher01
Copy link
Author

home-assistant_midea_ac_lan_2024-10-08T06-46-15.589Z.log

I also saw continuous on/off. The connection to the wifi should be OK because it's only 1 room away from the heater.

@chemelli74
Copy link
Collaborator

As first step please disable "custom_components/hon" as it generate a ton of "Detected blocking call...."

@chemelli74
Copy link
Collaborator

2024-10-08 09:45:59.614 DEBUG (Boiler) [midealocal.device] [142936511750394] Status update: {'power': True, 'mode': 'Energy-save', 'max_temperature': 20, 'min_temperature': 4, 'target_temperature': 10, 'current_temperature': 20, 'outdoor_temperature': 39.0, 'condenser_temperature': 22.5, 'compressor_temperature': 13.5, 'compressor_status': False}

Seems all good from log, maybe you are referring to F instead of C ?

@Punisher01
Copy link
Author

No. It's Celsius.

I raised this issue on older version also. Someone found a solution: georgezhao2010/midea_ac_lan#404 (comment)

Maybe this helps

@wuwentao
Copy link
Owner

wuwentao commented Oct 8, 2024

@Punisher01 could you help to share your device SN for us? then we can confirm the finally solution with your device type.

the error should be your device msg not match current code, we should add new type or new msg for your device to fix it.

@Punisher01
Copy link
Author

Punisher01 commented Oct 8, 2024 •
edited
Loading

0000CD311RSJRAC012A22B100101LT4G

image

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿

0000CD311RSJRAC012A22B100101LT4G

image

@wuwentao wuwentao changed the title Heat Pump Water Heater RSJRAC01 wrong temperature Feature(CD): Heat Pump Water Heater RSJRAC01 wrong temperature Oct 9, 2024
@wuwentao
Copy link
Owner

seems both mode should works for different device submodule.
maybe we can add a flag or switch for it, if the temperature is error, we can turn on or turn off it to confirm the result.

@BenLeitch4654
Copy link

Hello, I'm having the same issue, Temperatures aren't showing correct values... Was there a fix?

@wuwentao
Copy link
Owner

@Punisher01 @BenLeitch4654 I will add a customize switch/flag for your device,and I want to know your device subtype.
is this possible for you to provide your device config in your HA device.
for @Punisher01 device, from your debug log, your device id is 142936511750394
just execute this command in your HA device: cat /config/.storage/midea_ac_lan/142936511750394.json

if you have different device id, just replace it and provide me your device id json config file, it should include your device subtype info.
then I can add these subtype to auto detect mode without any action or flag changes.

thanks

@Punisher01
Copy link
Author 

  "name": "Boiler",
  "device_id": 142936511750394,
  "type": 205,
  "protocol": 3,
  "ip_address": "192.168.0.103",
  "port": 6444,
  "model": "RSJRAC01",
  "subtype": 0,
  "token": "5e86ed46d24a496bcf905e0bcbc965449679406c4d8de0c607379025bdfb039bed0469470c145f6bd9dd784e35868d7b819b463d7a96ddf36eabe30f4da22eff",
  "key": "bcb4a2be43a34f2d8ec0a5a961bedefc84d838b15be6441c9c9b50db8aece3f2"

@wuwentao wuwentao mentioned this issue Dec 16, 2024
Closed
@wuwentao wuwentao changed the title Feature(CD): Heat Pump Water Heater RSJRAC01 wrong temperature Bug(CD): Heat Pump Water Heater RSJRAC01 wrong temperature Dec 16, 2024
@BenLeitch4654
Copy link

Hi wuwentao

Thanks for your help, please see my config. Hope this helps

config_entry-midea_ac_lan-01JBDP2HW0Z4RAB7M6N2CNSBVT (1).json

@wuwentao
Copy link
Owner

@Punisher01 @BenLeitch4654 thanks for your configs, seems both of our subtype is 0, I'm not sure with old device subtype.
so i can't use device subtype to match both old and new device.
anyway, in order to keep old device and our new device can works in different mode and show different value, I will try to add a customize flag, then we can match in different if/else with this manual config.

@BenLeitch4654
Copy link

BenLeitch4654 commented Dec 18, 2024 via email

@wuwentao wuwentao added this to the 0.6.6 milestone Dec 19, 2024
@wuwentao wuwentao mentioned this issue Dec 25, 2024
Merged
@wuwentao
Copy link
Owner

wuwentao commented Dec 25, 2024 •
edited
Loading

@Punisher01 @BenLeitch4654

add some debug code in this PR: midea-lan/midea-local#345
you can manual change the midealocal python pip lib source code to test with it.

brief step:

  1. login to HAOS
  2. start homeassitant docker
  3. replace file for midealocal python pip package in homeassitant core(use wget to download github PR raw file and replace it)
  4. enable debug log for both midea_ac_lan and midealocal in configuration.yaml
  5. reboot HA.

how to change it, below is the detail steps:

  1. if you are using HAOS, you should install Add-on [Advanced SSH & Web Terminal] and disable [Protected Mode], once it done, you can ssh to your HAOS, and enter docker in HAOS docker exec -it homeassistant /bin/bash, contiune the step to check midea-local install path
  2. if you not using HAOS, just check midea-local install path as the next step
    3.get midea-local install path with command pip show midea-local, it will show info for you, for example, my location is Location: /usr/local/lib/python3.12/site-packages
  3. then we can replace the file under /usr/local/lib/python3.12/site-packages/midealocal/devices/cd/ with the code changes, using vi command line tool or wget to download github raw file.
  4. make sure you have enabled debug log in configuration.yaml
  5. reboot HA and test with it, confirm with the result or debug log.

after device reboot 3-5 minutes, download full debug log to your PC and upload this log file to current github issue
I will continue improve it with the debug log output.

in addition, default is { "lua_protocol": auto } I have process it for your device with subtype and model, if something can't works, you can set customize option with { "lua_protocol": new }.

old device will use { "lua_protocol": old }.

this is the first draft version, it may include error, especially for fahrenheit or celsius switch, I'm not sure with device response and current result, need to check the detail via debug log.

@wuwentao wuwentao linked a pull request Dec 26, 2024 that will close this issue
feat(cd): add water_level and temperature option with customize lua_protocol midea-lan/midea-local#345
Merged
@wuwentao
Copy link
Owner

@Punisher01 @BenLeitch4654 any updates?

@BenLeitch4654
Copy link

BenLeitch4654 commented Dec 31, 2024 via email

@julianjosias
Copy link

I have the same issue. If someone can explain how to do this I can test for you.

@wuwentao
Copy link
Owner

I have the same issue. If someone can explain how to do this I can test for you.

the detail steps has been post.
and also can be found in readme document with debug document.

@julianjosias
Copy link

Hi wuwentao, is this correct?
config_entry-midea_ac_lan-30c54274be3917a9b39bb6660eb613b1.json

@wuwentao
Copy link
Owner

@julianjosias NO, you should enable SSH for HAOS.
manual replace midealocal python3 pip package source code with latest github PR midea-lan/midea-local#345
I have post all the detail steps and action in previous post.

if you can't understand it, you can't finished this test and debug process.

anyway, think you very much for your, we can wait for some one can do it.

@Punisher01
Copy link
Author

Hello and sorry for late answer. I was on vacation. It seems it's working now

image
home-assistant_midea_ac_lan_2025-01-10T11-53-09.497Z.log

@wuwentao
Copy link
Owner

@Punisher01 thank you very much for your test result update.
I will try to merge it in next release, if there is any new bug or issue exist, we still can continue improve and fix it in future.

@Punisher01
Copy link
Author

Punisher01 commented Jan 13, 2025 via email

@wuwentao
Copy link
Owner

wuwentao commented Jan 13, 2025 •
edited
Loading

@Punisher01 it should be different issue.
need debug log to confirm the result for both of the issues.

please description the details with debug log, what you did, and what's the error, also include expected result.

@BenLeitch4654
Copy link

BenLeitch4654 commented Jan 13, 2025 via email

@wuwentao
Copy link
Owner

@BenLeitch4654 please help to test and share the result.
the detail steps has been post in previous message.
thank you very much!

@BenLeitch4654
Copy link

BenLeitch4654 commented Jan 14, 2025 via email

@wuwentao
Copy link
Owner

@BenLeitch4654 please refer to README document about debug and test
if you still can't enable it, just ingnore and we can skip it. thanks

@wuwentao
Copy link
Owner

please check with latest v0.6.6 release, it should include the fix.
if there is any issue exist, please upload your debug log file, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
0.6.6
Development

Successfully merging a pull request may close this issue.

feat(cd): add water_level and temperature option with customize lua_protocol midea-lan/midea-local
6 participants
@wuwentao @Punisher01 @chemelli74 @Issues-translate-bot @julianjosias @BenLeitch4654