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

MHI-AC-Ctrl v2.6 update? #51

Closed
JoepWi opened this issue Jan 29, 2023 · 13 comments
Closed

MHI-AC-Ctrl v2.6 update? #51

JoepWi opened this issue Jan 29, 2023 · 13 comments

Comments

@JoepWi
Copy link

JoepWi commented Jan 29, 2023

Can we incorporate all updates from this version?: https://github.com/absalom-muc/MHI-AC-Ctrl/releases/tag/v2.6

@Kurisutian
Copy link

Any information when there will be an upgrade? Especially the changes around the external thermostat and the fan auto mode are great improvements. 😃

@Bill-Dung
Copy link
Contributor

The main problem will be that ESPHome only supports three fan speeds + auto, but the MHI ACs have four speeds and auto.

@Kurisutian
Copy link

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).
However, checking it out and using that code did not work for me.

@Bill-Dung
Copy link
Contributor

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.

@ervee
Copy link

ervee commented Mar 3, 2023

What about this solution: #22 (comment)

@Bill-Dung
Copy link
Contributor

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.

@Bill-Dung
Copy link
Contributor

Find it here: https://github.com/Bill-Dung/MHI-AC-Ctrl-ESPHome
AC has now 5 different fan modes but I'm not sure if the auto mode works proper, keep testing.

@Kurisutian
Copy link

Find it here: https://github.com/Bill-Dung/MHI-AC-Ctrl-ESPHome AC has now 5 different fan modes but I'm not sure if the auto mode works proper, keep testing.

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?

@Bill-Dung
Copy link
Contributor

Yes, it's 2.5

@ervee
Copy link

ervee commented Mar 6, 2023

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.

@Bill-Dung
Copy link
Contributor

Bill-Dung commented Mar 7, 2023

CLIMATE_FAN_QUIET is not be showing up in Home Assistant Card like CLIMATE_FAN_MIDDLE and CLIMATE_FAN_FOCUS.

@RobertJansen1
Copy link
Collaborator

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.

#54

testing can be done from https://github.com/RobertJansen1/MHI-AC-Ctrl-ESPHome

@JoepWi JoepWi closed this as completed Jul 14, 2023
@serrj-sv
Copy link
Contributor

serrj-sv commented Aug 4, 2023

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.

#54

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:
#62

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

No branches or pull requests

6 participants