-
Notifications
You must be signed in to change notification settings - Fork 33
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
Feature(C3): (Heat Pump Wi-Fi Controller) - add new parameters to entities #404
Comments
@michalszym sorry, we can't add all parameters,as different parameters have different units/type. in addition, I will also try to add some basic and easy to do parameters. thanks |
@fagallo
in addition ,it should be the feature requirement for current issue, need to got some debug log and confirm the result. how to change it, below is the detail steps:
if you want to check debug log output with your device display name and value. |
@micturkey @fagallo @AndreasTUHU @erikgieseler @bdecl If you are interested in these C3 device params, you can test this draft PR, verify the output and give feedback with what you expected params, i will try to add some with it. |
Hi,
|
Hi, In att some logs. I also getting unsupported protocol for MessageQueryAppliance, MessageQueryDisinfect and MessageQueryUnitPara. Do we also has to share the firmware version of the Controller ? |
@bdecl @erikgieseler thanks for your info update.
as this is a draft test version, so I have add 3 new query: in addition, all these query expected a response msg from device, if we can't recv msg, we can do nothing. |
Hi, |
Hi, home-assistant_midea_ac_lan_2024-12-19T23-02-48.325Z.log Firmware versions visible on Heatpump controller Midea C3 Smart Home detected Firmware V.0521318_0000 |
@bdecl @erikgieseler thanks. seems both of our device only support QueryECO and got a Unidentified protocol(expected result, we have recvied response and not parse it)
could you help to test the latest changes with your device? and provide the full debug log? |
Hi, I tried this. Log |
@restudio thank you very much for your test and debug log. if device can't response to our query message, also not send these update message to our HA device, we can't support these params. |
1. current disinfect is the same as tbh, fix tbh set error 2. add disinfect query 3. TO BE DONE: set disinfect, as it's a switch 4. add some addition attrs fix wuwentao/midea_ac_lan#399 and wuwentao/midea_ac_lan#404 maybe need a new PR for it.
sorry, I have add all these values to current v0.6.6 in midea-local lib side with debug log output, NOT available in midea_ac_lan, as our devices can't response to these new queries, we can't got these params value. so I have closed current issue, anyway, there is still some C3 device issue opening, I will continue improve it if there is any new updates. |
Sorry for such a long time without response, but I had health problems. |
@michalszym thanks for these info share. |
As far as I know to access the modbus registers, you need to use the RS-485 port of the wired controller. |
@AndreasTUHU it's another solution and can't be use for current integration. |
Device type and model (or SN)
Heat Pump Wi-Fi Controller 171H120F (0), 0000C3310171H120F28044101688DYU8
The detail description of new feature
In reference to the discussion in the thread: #387, please add all parameters that can be read from the heat pump wi-fi controller, so that each integration user can decide which of them they need .
I am attaching photos of the controller with available parameters to the application.
I don't know if it will help, but in one of the manuals for the Midea Heatpump WiFi controller that I found, the modbus parameters are described in detail.
The manual can be found here: https://kaisai.com/pl/files/page_files/464/kaisai-arctic-power-instrukcja-sterownika-pl-v02.pdf and information about modbus starts on page 48 (41).
The text was updated successfully, but these errors were encountered: