Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

Manage digits in target temperature for Climate Devices #61

Merged
merged 1 commit into from
Jan 29, 2021

Conversation

ollo69
Copy link
Contributor

@ollo69 ollo69 commented Dec 12, 2020

This PR contain 2 fixes for climate device:

  1. add list of allowed temperature steps to use in HA and fix the set_temperature method to manage digits: this will allow in HA to define different temperature steps and will manage steps with decimals. This PR replace PR climate: Add halves when divider is at least 2 #54 and PR Add support for decimal values on set temperature #59

  2. add a parameter to set_temperature method to choose update mode (fix for issue Siterwell GS361A-H04 #51)

I will create required PR in HA to bump new version and apply required changes if this PR will be approved.

- added list of allowed temperature step to use in HA
- fixed set_temperature to manage digits and choose update mode
Copy link

@matitalatina matitalatina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this code and it works with my thermostat BHT-002-GCLW

@matitalatina
Copy link

@PaulAnnekov please, can you have a look at it? We can't proceed further otherwise.

@PaulAnnekov PaulAnnekov merged commit 302ec3e into PaulAnnekov:master Jan 29, 2021
@PaulAnnekov
Copy link
Owner

@matitalatina @ollo69 thank you. Already in version 0.0.10.

@ollo69
Copy link
Contributor Author

ollo69 commented Jan 29, 2021

Thanks @PaulAnnekov, I will create HA PR this w.e.

@ollo69 ollo69 deleted the climate-decimal-steps branch January 30, 2021 02:38
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.

3 participants