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

Added MIDEA/KOMECO IRHVAC protocol #3227

Closed
wants to merge 1 commit into from

Conversation

lvta0909
Copy link

Added support to MIDEA/KOMECO HVAC protocol.
Its somewhat similar to Mitsubishi hvac protocol.

More information about the protocol / source code reference:
http://veillard.com/embedded/midea.html
https://github.com/sheinz/esp-midea-ir/blob/master/midea-ir.c

@ascillato2
Copy link
Collaborator

Hi,

Thanks a lot for sharing your ideas.

As Tasmota can't handle ALL possible IR protocols, due to flash size restrictions, we should be better implementing a RAW IR send/receive as requested at #2116.

By now, This PR is not going to be merged due to be outside the actual scope of Tasmota. Please, if you have further questions or ideas, do not hesitate on contacting us in the Tasmota Discord Chat. Thanks.

@ascillato2 ascillato2 closed this Oct 1, 2018
@Zuikkis
Copy link

Zuikkis commented Feb 7, 2019

I'd like to disagree on this decision.

Tasmota already has many "unnecessary" extensions, like tons of different sensors which are not useful for most people. But it's great to have them in case you need them! They can be enabled and disabled in the config when building, when disabled they don't take up space.

IR_HVAC is default disabled in my_user_config.h, so adding Midea does not take any space.

RAW IR send/receive does not really help for air conditioners, because generally they always have huge IR data packets which cannot be received (or sent?) unless you recompile Tasmota with increased buffer sizes, as explained by many users in #2116..

I have this Wifi IR blaster:
https://github.com/arendst/Sonoff-Tasmota/wiki/YTF-Wifi-Smart-IR---Tuya-Universal-Smart-IR-Hub---TYWE3S-module

Recent Tasmota supports it. With such device it would be great to have a Tasmota firmware that supports as wide range of IR devices as possible, but really is no need to have sensor support etc..

So just let users build what they want. :)

I modified the Midea commit to fit current development branch, and it seems to work nicely. Thanks @lvta0909 !

@no-Legend
Copy link

@Zuikkis
I own two mobile hvac's from media (in Germany the company sell as comfee or koenic)
I tried to use the raw command, but I was not able to get the device controlled.
Raw was not working in any kind I tried it.

Is there a way to integrate the midea code in the latest master?

@arendst
Copy link
Owner

arendst commented Jul 21, 2019

Working on it now.

@arendst arendst self-assigned this Jul 21, 2019
arendst added a commit that referenced this pull request Jul 21, 2019
Add support for optional IRHVAC Midea/Komeco protocol (#3227)
@arendst arendst removed their assignment Jul 21, 2019
@arendst
Copy link
Owner

arendst commented Jul 21, 2019

Done

@no-Legend
Copy link

@arendst
I tested now the protocol for midea. Works good for my midea (comfee,koenic) hvac.

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

Successfully merging this pull request may close these issues.

5 participants