You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In case of wuwentao/midea_ac_lan (Midea AC LAN) I needed to enter my server credentials (SmartHome app in my case) and it could also extract tokens without any problems (despite the warnings in the Readme, BTW)
same for this homeassistant-midea-air-appliances-lan, I also need to enter credentials (which currently fails, see above)
FYI, interestingly, while the (device) ID is the same, the tokens (and key) are indeed different
Also, BTW, the SmartHome app shows and asks for me to set a password (for the device, I assume, my account obviously has one). I don't know, if I had done that, this might have changed the result. (It's only Bluetooth IIRC)
Also:
wuwentao/midea_ac_lan (Midea AC LAN) apparently seems to support much more devices/device types (I personally do not need that, though)
wuwentao/midea_ac_lan (Midea AC LAN) is not that localized in German at least
wuwentao/midea_ac_lan (Midea AC LAN) needs to be configured to show more sensors etc., but that is also not a big problem
For example [a problem is], having two Home Assistant instances accessing same device, or using one of other Midea appliance integrations in combination with this one. To avoid problems, use a single integration - this one 🙂.
What makes this better? 😉
I assume running them side-by-side makes little sense?
Also, could not you maybe work together? Instead of each one maintaining their own fork? Especially, if this integration could at some point maybe get into HAOS core, instead of HACS?
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
Hi @nbogojevic and thanks a lot for your work!
I have tried out your integration, but it currently fails, because of #238 (comment). However, I also found https://github.com/wuwentao/midea_ac_lan and https://github.com/mill1000/midea-ac-py/ – a fork by someone else. They also seem to be based on Both seem to be maintained, too.
Testing
I installed the other two side-by-side and, as you can see, they basically work:

The only major difference I see is the setup process:
mill1000/midea-ac-py(Midea Smart AC), it could find and connect to the device directly and extract some token and key, somehowwuwentao/midea_ac_lan(Midea AC LAN) I needed to enter my server credentials (SmartHome app in my case) and it could also extract tokens without any problems (despite the warnings in the Readme, BTW)homeassistant-midea-air-appliances-lan, I also need to enter credentials (which currently fails, see above)key) are indeed differentAlso, BTW, the SmartHome app shows and asks for me to set a password (for the device, I assume, my account obviously has one). I don't know, if I had done that, this might have changed the result.(It's only Bluetooth IIRC)Also:
wuwentao/midea_ac_lan(Midea AC LAN) apparently seems to support much more devices/device types (I personally do not need that, though)wuwentao/midea_ac_lan(Midea AC LAN) is not that localized in German at leastwuwentao/midea_ac_lan(Midea AC LAN) needs to be configured to show more sensors etc., but that is also not a big problemApart from that, I see no major differences.
Questions
Given you confidently state in your Readme:
What makes this better? 😉
I assume running them side-by-side makes little sense?
Also, could not you maybe work together? Instead of each one maintaining their own fork? Especially, if this integration could at some point maybe get into HAOS core, instead of HACS?
References/Backlinks
Beta Was this translation helpful? Give feedback.
All reactions