-
Notifications
You must be signed in to change notification settings - Fork 22
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
Fix: Midea heat pump outside temperature issue when below zero #22
Conversation
Fix: Midea heat pump outside temperature issue when below zero #427
Co-authored-by: Simone Chemelli <simone.chemelli@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Although there is a linting error to be fixed.
@erikgieseler
|
we can accept the changes, and port it to midea-local. |
The action errors are fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Just a small question: is that valid only for "c3" devices or there is something similar for others ?
Unfortunately, I can't answer that because I only have a C3 heat pump. Perhaps the same problem also exists with other devices where the exact same old code was used. |
We need to rework the code and remove all those duplicates. |
I have done a search with the old code across all other devices. |
Fix: Midea heat pump outside temperature issue when below zero georgezhao2010/midea_ac_lan#427