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

An error when change mode. #186

Open
djlukas7 opened this issue Jul 30, 2022 · 4 comments
Open

An error when change mode. #186

djlukas7 opened this issue Jul 30, 2022 · 4 comments
Labels
bug Something isn't working

Comments

@djlukas7
Copy link

Describe the bug (描述一下问题)
The error occurs when changing the state from off to heating / cooling.
The only visible mode of operation is off. The other modes are invisible but I can select it and then I get the error visible on sreenshot.
The air conditioner is Hyundai.

Screenshots / Logs / Pcap File(屏幕截图/日志/抓包文件)
obraz

obraz

Versions (版本信息)

  • Home Assistant version: 2022.7.6
  • Midea msmart version: v0.2.3
@djlukas7 djlukas7 added the bug Something isn't working label Jul 30, 2022
@mill1000
Copy link

This is due to mac-zhou/midea-msmart#90

You can manually downgrade the dependency before the breaking commit. Be sure to run this in the container and/or virtual-env that HA runs in.

pip install git+https://github.com/mac-zhou/midea-msmart@29522b209d9dcd1775d0d57f9d27214d65e339c9

@djlukas7
Copy link
Author

The same error, version 0.2.4 was uploaded

obraz

@mill1000
Copy link

Did you install the commit I recommended or the 0.2.4 tag?

Look at line 332 and 336 in appliance.py. They should read

return IntEnumHelper.names(self._supported_op_modes)

and

return IntEnumHelper.names(self._supported_swing_modes)

@djlukas7
Copy link
Author

I edited this file manually and changed these lines. Everything works now. Thank you very much for help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants