mac-zhou / midea-ac-py

Home Assistant Custom Integration for Midea Group(Hualing, Senville, Klimaire, AirCon, Century, Pridiom, Thermocore, Comfee, Toshiba, Carrier, Goodman, Friedrich, Samsung, Kenmore, Trane, Lennox, LG and much more) Air Conditioners via LAN.
MIT License
539 stars 103 forks source link

Manifest requires msmart 0.2.4 which is not available in PyPi #183

Open mill1000 opened 2 years ago

mill1000 commented 2 years ago

The current manifest requires msmart 0.2.4 which is not available in PyPi. This is causing install failures for users (reported in #182) because the dependency can not be met.

https://github.com/mac-zhou/midea-ac-py/blob/a1699f714e4113ccd25f8f32f2d245781ce654fc/custom_components/midea_ac/manifest.json#L7

We should bump the manifest version to 0.2.5 or merge #178 to support multiple versions.

AndoKarim commented 2 years ago

I bumped the manifest version and the behavior with my AC is totally different. I don't have all the controls when using the classic card. I got those errors when trying to interract : "error set_hvac_mode"

mill1000 commented 2 years ago

That is likely because of mac-zhou/midea-msmart#90. Unfortunately this integration is currently in a bit of a bad state.

I've got a branche with a number of fixes merged here: mill1000/midea-msmart.

To install via pip

pip install git+https://github.com/mill1000/midea-msmart.git
AndoKarim commented 2 years ago

So I'm using the latest version of midea-ac-py and your fork / branche and I still have those issues : image

and the sensors are at 0C image

This is my manifest : image

And if I add the configuration too in the configuration.yaml : image

mill1000 commented 2 years ago

To me it looks like HA is still using the old version of midea-msmart because the get_capabilities call is missing. I've had issues where HA doesn't update the requirements even if the manifest changes. Can you manually install the dependency via pip?

AndoKarim commented 2 years ago

pip is not working on my end, I replied to you on the other issue you're following. Maybe we can stick to this one?

image I cannot use any command in Terminal

AndoKarim commented 2 years ago

Update: I ended up discovering there is two Terminal modules. I installed the right one. I'm trying to pip install your branche

Update: Still the same error after doing pip install git+https://github.com/mill1000/midea-msmart.git@future

mill1000 commented 2 years ago

Great. I was just about to reply to see if you were using this add-on or other "official" one. The official one seems quite limited in its use

AndoKarim commented 2 years ago

Yeah I solved the ssh issue but still stuck with this msmart.

I removed the requirements from manifest.json. But still having the same issue, not finding _getcapabilities I even did pip uninstall msmart and reinstalled the 0.2.4, tried the 0.2.5 official and your fork..

I am missing something but don't know why. It seems there is a sneaky msmart that is still installed somewhere and grabbing my midea-ac-py

mill1000 commented 2 years ago

If you're getting issues with get_capabilities, then it sounds like you're installing msmart...at least not where HA is looking for it.

Are you running these pip commands within the Docker container that runs HA?

AndoKarim commented 2 years ago

I am running the pip install from the root it seems

AndoKarim commented 2 years ago

Ok I fixed the issue here are the steps :

Open a terminal and enter in docker mode :

Everything came back in track.

The mistake here is to use Terminal and connect as root and do the steps. You need to be in the docker

Thanks a lot for your help.

mill1000 commented 2 years ago

No problem. Glad you got it working again.

If it's of interest to you. I also have a future branch of midea-ac-py here that add some additional features