Skip to content

Homematic Fixes#6769

Merged
pvizeli merged 4 commits into
home-assistant:devfrom
danielperna84:homematic
Mar 25, 2017
Merged

Homematic Fixes#6769
pvizeli merged 4 commits into
home-assistant:devfrom
danielperna84:homematic

Conversation

@danielperna84
Copy link
Copy Markdown
Contributor

Description:

For HomeMatic the used library has some fixes. One includes operational modes for select thermostats that were missing until now. So this is just as dependency-bump + small addition to the climate-platform for the new modes.

@pvizeli
Copy link
Copy Markdown
Member

pvizeli commented Mar 24, 2017

I think that inside doc stay that mode not include confort and ecco mode? Or is that only for IP devices?

@danielperna84
Copy link
Copy Markdown
Contributor Author

danielperna84 commented Mar 24, 2017

There aren't many thermostats that support eco and comfort, yes. But in pyhomematic only the relevant ones have these modes in their action-node, and if I read your climate-code correctly, only the modes present in the action-node will be used in HASS. So any thermostat that doesn't support them won't have them in the UI.

What happens with this change (f. ex with HM-CC-RT-DN) is:
Regardless of if you are in manual oder auto mode, selecting "Lowering" or "Comfort" from the UI switches the thermostat into that mode, indicated by either a sun- or moon-symbol. The general mode (auto/manual) however stays the same in the UI and on the thermostat. So in the UI "Comfort" or "Lowering" is never the active option, since these modes are more like a shortcut to quickly set the target-temperature to a pre-defined value, which doesn't change anything about the actual operational mode.

@danielperna84
Copy link
Copy Markdown
Contributor Author

HM-TC-IT-WM-W-EU had the wrong channels in 0.1.23. Made another version to address that issue.

@pvizeli pvizeli added this to the 0.41 milestone Mar 25, 2017
@pvizeli pvizeli merged commit 4470487 into home-assistant:dev Mar 25, 2017
@danielperna84 danielperna84 deleted the homematic branch March 25, 2017 12:00
@fabaff fabaff mentioned this pull request Apr 6, 2017
@home-assistant home-assistant locked and limited conversation to collaborators Jun 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants