KEMBAR78
Add AC mode to heat_control component by mtreinish · Pull Request #2719 · home-assistant/core · GitHub
Skip to content

Conversation

mtreinish
Copy link
Contributor

@mtreinish mtreinish commented Aug 3, 2016

Description:
Add an air conditioner mode to the heat_control component. The same concept and operation used for plug in heaters applies equally well to in-wall or window AC units except the logic is reversed.

Related issue (if applicable): fixes #

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#723

Example entry for configuration.yaml (if applicable):

thermostat:
  platform: heat_control
  name: Study
  heater: switch.study_heater
  target_sensor: sensor.study_temperature
  min_temp: 15
  max_temp: 21
  target_temp: 15
  ac_mode: True

Checklist:

If user exposed functionality or configuration variables are added/changed:

If code communicates with devices, web services, or a:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New dependencies are only imported inside functions that use them (example).
  • New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
  • New files were added to .coveragerc.

If the code does not interact with devices:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • Tests have been added to verify that the new code works.

This commit adds a new option to the heat_control component, ac_mode.
When set to true, this treats the toggle device as a cooler instead
of a heater. The concept being if you have a window or in-wall ac
unit that doesn't have a built-in thermostat having the home assistant
implemented thermostat would be as useful as for space heaters.

This commit adds a new option to the heat_control component, ac_mode.
When set to true, this treats the toggle device as a cooler instead
of a heater. The concept being if you have a window or in-wall ac
unit that doesn't have a built-in thermostat having the home assistant
implemented thermostat would be as useful as for space heaters.
@mtreinish
Copy link
Contributor Author

I also considered doing this as a new component, but I figured this approach was better than duplicating the entire module to just swap the comparisons.

mtreinish added a commit to mtreinish/home-assistant.io that referenced this pull request Aug 3, 2016
This commit adds the heat_control option add in change:
home-assistant/core#2719 to the documentation for that
component.
@balloob
Copy link
Member

balloob commented Aug 3, 2016

Looks good! 🐬

@balloob balloob merged commit 94acda2 into home-assistant:dev Aug 3, 2016
balloob pushed a commit to home-assistant/home-assistant.io that referenced this pull request Aug 3, 2016
This commit adds the heat_control option add in change:
home-assistant/core#2719 to the documentation for that
component.
@mtreinish mtreinish deleted the add_ac_mode branch August 3, 2016 05:09
@home-assistant home-assistant locked and limited conversation to collaborators Mar 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants