Added min_temp and max_temp configuration options #209
Conversation
…mps supporting stepping temperature down to 8 degrees instead of separate 8 degrees mode.
custom_components/gree/climate.py
Outdated
| @@ -318,6 +323,7 @@ def FetchResult(self, cipher, ip_addr, port, timeout, json): | |||
| clientSock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM) | |||
| clientSock.settimeout(timeout) | |||
| clientSock.sendto(bytes(json, "utf-8"), (ip_addr, port)) | |||
| _LOGGER.info('3') | |||
There was a problem hiding this comment.
My bad, we do not, seems I did not read my own pull request well enough. This was some leftover debugging code from some previous codebase or something, no need for it. Amended my pull request and removed it.
|
Hey @jarpatus i think we need to update this PR for it to be able to integrate. Let me know if thats possible and i'll make sure to test & merge it a.s.a.p. |
|
@jarpatus @RobHofmann currently my gree cosmo units will continue to cool down room waaay beyond target temp (I set 21, room was chilled to 18). I use external entity to send current room temp to AC and it works I think. |
No this is about the minimum & maximum value the temperature of the device can be set to. Thats unrelated to "sensing the real tempearture in the room". |
|
Just noticed that we also need to make this compatible with the Fahrenheit release. |
Added min_temp and max_temp configuration options due to some heat pumps supporting stepping temperature down to 8 degrees instead of separate 8 degrees mode i.e. Innova IWZA09NI-1.