Giter Club home page Giter Club logo

Comments (5)

RobHofmann avatar RobHofmann commented on August 16, 2024

Merged & Closed.

Released in 2.1.0. Can be updated through HACS.

from homeassistant-greeclimatecomponent.

wacher74 avatar wacher74 commented on August 16, 2024

Hi,

Thank you for support 8C :)

Here is a suggestion, just a little fix (sorry I don't know github enough)

def UpdateHATargetTemperature(self):
    # Sync set temperature to HA
    if (int(self._acOptions['StHt']) == 1):
        self._target_temperature = 8
    else:
        self._target_temperature = self._acOptions['SetTem']
    _LOGGER.info('HA target temp set according to HVAC state to: ' + str(self._target_temperature))

Reason: AC reports the temperature what is set for normal heating (ex. 22). Gree+ app, A/C display shows 8C, according to reality.

from homeassistant-greeclimatecomponent.

PLTorrent avatar PLTorrent commented on August 16, 2024

Hi @wacher74, this is what I was missing. Will test if your proposed solution works as intended and if yes then I'll manage the github process.

In the comments to my pull there is:

"The only thing missing is that when this mode is activated it would be good to display target temp as 8 deg C the same as on the device display. Maybe someone could add this or give me a hint how to do it quickly. Didn't want to mess up the temp refreshing/setting."

Will do my best to verify tomorrow.

from homeassistant-greeclimatecomponent.

wacher74 avatar wacher74 commented on August 16, 2024

Thank you for your help!
I just added an if statement to UpdateHATargetTemperature()
If StHt is active then 8 is reported to HomeAssistant instead of SetTem (which is temp for normal heating)

from homeassistant-greeclimatecomponent.

PLTorrent avatar PLTorrent commented on August 16, 2024

Just tested. Actually your solution is very clever one. I was afraid of some strange things happening with thermostat control since the range for this control is 16-30 deg C. However it all works really nice, since when AC is working in 8degC mode it ignores any input changing temp. Also clicking on the arrows of the thermostat only changes the temp for a couple of seconds and than it's all back to normal. I was afraid that changing temp might turn off the 8C mode.

Since this is a very nice solution I will make a pull request with your code.

Thank you for your support!

from homeassistant-greeclimatecomponent.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.