-
Notifications
You must be signed in to change notification settings - Fork 39
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
MHI-AC-Ctrl v2.6 update? #51
Comments
Any information when there will be an upgrade? Especially the changes around the external thermostat and the fan auto mode are great improvements. 😃 |
The main problem will be that ESPHome only supports three fan speeds + auto, but the MHI ACs have four speeds and auto. |
So basically you're saing that it won't be possible to upgrade to 2.6 unless this shortcoming has been addressed to and fixed by the ESPHome team? Just wondering as it seems that someone did just used the code, created a component to integrate in ESPHome and upgraded to 2.6 (https://github.com/tfyoung/esphome-MHIHeatPump). |
I didn't say it's impossible, you have to create a component and extend the climate class, this cannot be done on passing by. The mentioned component didn't work for me either, it seems to be cobbled together quickly. |
What about this solution: #22 (comment) |
Sounds interesting, I'll give a try next week. If it works I could create a fork of MHI-AC-Ctrl-ESPHome so everyone can use it until ginkage makes an update available. |
Find it here: https://github.com/Bill-Dung/MHI-AC-Ctrl-ESPHome |
Thanks, I will test it. Is the header file the only thing which changed compared to this repo? So it would be v2.5, correct? |
Yes, it's 2.5 |
I will also look into this now I see how this change is done. Thank you. And if it works I or Bill-Dung or someone could make a pull request. Perhaps using the CLIMATE_FAN_QUIET instead of DIFFUSE. CLIMATE_FAN_QUIET should also be available in climate and esphome and is perhaps a more logical choice for ultra-low fan speed. |
CLIMATE_FAN_QUIET is not be showing up in Home Assistant Card like CLIMATE_FAN_MIDDLE and CLIMATE_FAN_FOCUS. |
CLIMATE_FAN_QUIET is now implemented in home assistant: home-assistant/core@d14000f i created a pull request implementing both v2.7R4 and the fan modes from @Bill-Dung, adjusting the DIFFUSE to Quiet. testing can be done from https://github.com/RobertJansen1/MHI-AC-Ctrl-ESPHome |
I don't like the fact that "quiet" comes after "high" in Dashboard card. I propose to change this a bit, see: |
Can we incorporate all updates from this version?: https://github.com/absalom-muc/MHI-AC-Ctrl/releases/tag/v2.6
The text was updated successfully, but these errors were encountered: