Devotics / heatzy-home-hassistant

Climate Home Assistant component for Heatzy Pilot
MIT License
24 stars 20 forks source link

Heatzy component broken with HA 0.96 #4

Closed benaisn closed 5 years ago

benaisn commented 5 years ago

Hey there Probably due to the last release of HA 0.96 (with a new climate configuration, see https://developers.home-assistant.io/blog/2019/07/03/climate-cleanup.html), the Heatzy component doesn't work anymore

Dramloc commented 5 years ago

Hello,

Thank you for letting me know ! It seems like Home Assistant API is getting closer to Heatzy API, so it should not be too hard to fix. I will look into this.

PR welcomed if you want to 😃

Dramloc commented 5 years ago

Hello,

The fix should be implemented on the fix/0.96-compatibility branch. Not sure if the behavior is perfect yet. The component looks like this now:

image

"Opération" (hvac_mode) is required by HA and is always set to HVAC_MODE_AUTO. "Préréglage" (preset_mode) can be changed to change the Heatzy status.

Can you try this fix and let me know what you think?

Installation script for fix/0.96-compatibility:

mkdir -p ~/.homeassistant/custom_components
cd ~/.homeassistant/custom_components
curl -L https://api.github.com/repos/Devotics/heatzy-home-hassistant/tarball/fix/0.96-compatibility | tar -xz
cp -rl Devotics-heatzy-home-hassistant-*/heatzy .
rm -rf Devotics-heatzy-home-hassistant-*
benaisn commented 5 years ago

Hey Antoine Just tried and it works well ! well done :) Just a small delay when updating the status (preset_mode) from / to Heatzy, but not a big deal, and maybe that's my configuration

On Mon, Jul 29, 2019 at 10:08 PM Antoine Colmard notifications@github.com wrote:

Hello,

The fix should be implemented on the fix/0.96-compatibility branch. Not sure if the behavior is perfect yet. The component looks like this now:

[image: image] https://user-images.githubusercontent.com/4021308/62078454-fc337480-b24c-11e9-8a68-320fbf7d6061.png

"Opération" (hvac_mode) is required by HA and is always set to HVAC_MODE_AUTO. "Préréglage" (preset_mode) can be changed to change the Heatzy status.

Can you try this fix and let me know what you think?

Installation script for fix/0.96-compatibility:

mkdir -p ~/.homeassistant/custom_components cd ~/.homeassistant/custom_components

curl -L https://api.github.com/repos/Devotics/heatzy-home-hassistant/tarball/fix/0.96-compatibility | tar -xz

cp -rl Devotics-heatzy-home-hassistant-*/heatzy .

rm -rf Devotics-heatzy-home-hassistant-*

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Devotics/heatzy-home-hassistant/issues/4?email_source=notifications&email_token=AINCKCVNHAGDLMXROH2HOV3QB5E5TA5CNFSM4IGY5BF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3B3O7A#issuecomment-516142972, or mute the thread https://github.com/notifications/unsubscribe-auth/AINCKCTJ7YJMO535ORLMGYDQB5E5TANCNFSM4IGY5BFQ .

-- Nicolas Benais

Dramloc commented 5 years ago

Thanks, I will close the issue and merge into master.

I also have a bit of delay when using the component. If I recall correctly, the Heatzy API has a small delay when you update device status. This might be the cause but I haven't found a easy fix yet.