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

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

Open
mill1000 opened this issue Jul 26, 2022 · 12 comments
Open

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

mill1000 opened this issue Jul 26, 2022 · 12 comments

Comments

@mill1000
Copy link

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.

"requirements": ["msmart==0.2.4", "pycryptodome", "pycryptodomex", "click"],

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

@AndoKarim
Copy link

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
Copy link
Author

mill1000 commented Jul 29, 2022

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
Copy link

AndoKarim commented Jul 29, 2022

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
Copy link
Author

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
Copy link

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
Copy link

AndoKarim commented Jul 29, 2022

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
Copy link
Author

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
Copy link

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 get_capabilities
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
Copy link
Author

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
Copy link

I am running the pip install from the root it seems

@AndoKarim
Copy link

Ok I fixed the issue here are the steps :

Open a terminal and enter in docker mode :

  • docker exec -it homeassistant bash (you need to disable protected mode from Terminal module)
  • Then do a pip uninstall msmart (I had version 0.2.3 installed)
  • I did a pip install of the 0.2.4 commit of mac-zhou

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
Copy link
Author

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

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

2 participants