unixko / MitsuCon

Mitsubishi Heat Pump Controller for Home Assistant
Apache License 2.0
68 stars 12 forks source link

C F switch #5

Open IsaacBreuer opened 4 years ago

IsaacBreuer commented 4 years ago

Is there a way to change it to farenheit ? The original heatpump has a property of iscelcius , but it's not included in this current native or classic mode, how to I go about farenheit?

unixko commented 4 years ago

Did you change HA unit system in configuration.yaml ?

homeassistant:
  unit_system: imperial

Also please test your IR remote control that what is the lowest and highest temperature number and step size. Then try to update those values in mitsucon.h configuration file. If it works please post your IR remote control picture and your HA climate screenshot for future reference.

IsaacBreuer commented 4 years ago

So in mitsucon.h I set the low high values in c or f, when unit_system: is imperial

On Sun, Sep 22, 2019, 5:13 AM unixko notifications@github.com wrote:

Did you change HA unit system in configuration.yaml ?

homeassistant: unit_system: imperial

Also please test your IR remote control that what is the lowest and highest temperature number and step size. Then try to update those values in mitsucon.h configuration file. If it works please post your IR remote control picture and your HA climate screenshot for future reference.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/unixko/MitsuCon/issues/5?email_source=notifications&email_token=ABNS2YSGP7SCR57FSJMKA23QK4ZKBA5CNFSM4IZBPGPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7JB2XQ#issuecomment-533863774, or mute the thread https://github.com/notifications/unsubscribe-auth/ABNS2YQB7FDZMQHYXUAD6FTQK4ZKBANCNFSM4IZBPGPA .

unixko commented 4 years ago

Yes please set in F like 60-88. I hope HA will handle all C/F conversion and use deterministic unit from value range.

kenberland commented 4 years ago

I also have this issue. When used in conjunction with SwiCago's custom components, discovered climate sensors using this repo report in Celsius. It appears that they need to be converted before being sent.

unixko commented 4 years ago

I tested and found that when changing HA Unit System from metric to imperial, other Climate devices can switch unit on the fly but not for MQTT HVAC. That is because MQTT HVAC component doesn't support temperature_unit parameter. Pull request to HA like below is needed.

https://github.com/home-assistant/home-assistant/pull/30602

When MQTT HVAC and MQTT Discovery are supported I will add temperature_unit = TEMP_CELSIUS to MitsuCon code.

unixko commented 4 years ago

@yabuts @kenberland Home Assistant 0.109.0 added support to change unit in MQTT Climate to Fahrenheit. Please try my latest PR https://github.com/unixko/MitsuCon/commit/5650ccb9fa10562699c86ced657005adac915251. Does it work well?

unixko commented 4 years ago

@dzungpv @gysmo38 you may be interested to utilize HA 0.109.0 MQTT HVAC variable temperature_unit and let HA handles all of conversion instead.