Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Impossible to set target temps #13

Open
kevindebosscher opened this issue Jan 2, 2024 · 9 comments
Open

Impossible to set target temps #13

kevindebosscher opened this issue Jan 2, 2024 · 9 comments

Comments

@kevindebosscher
Copy link

Hi, I added as per documentation.
Dallas sensor is measuring actual temp in the room.
However not possible to set actual target temps...

Just me or did I overlooked something?

Thx,
Kevin
issues ac GREE

@Hagakurje
Copy link
Owner

Don't know, my ac looks good. Is it default ac card?
Screenshot 2024-01-02 220244

@Oleg-Sob
Copy link

After compilation, there is also no way to set the temperature, since it is set to 0. In any other climate control (for example, in Better Thermostat) you raise the temperature above 16 and everything appears immediately.

@Oleg-Sob
Copy link

During compilation it gives an error

In file included from src/main.cpp:43:
src/gree_ir.h: In member function 'virtual void GreeAC::control(const esphome::climate::ClimateCall&)':
src/gree_ir.h:89:14: warning: enumeration value 'CLIMATE_MODE_AUTO' not handled in switch [-Wswitch]
89 | switch (climateMode)
| ^
src/gree_ir.h:133:14: warning: enumeration value 'CLIMATE_FAN_ON' not handled in switch [-Wswitch]
133 | switch (fanMode)
| ^
src/gree_ir.h:133:14: warning: enumeration value 'CLIMATE_FAN_OFF' not handled in switch [-Wswitch]
src/gree_ir.h:133:14: warning: enumeration value 'CLIMATE_FAN_MIDDLE' not handled in switch [-Wswitch]
src/gree_ir.h:133:14: warning: enumeration value 'CLIMATE_FAN_DIFFUSE' not handled in switch [-Wswitch]
src/gree_ir.h:133:14: warning: enumeration value 'CLIMATE_FAN_QUIET' not handled in switch [-Wswitch]
src/gree_ir.h:164:14: warning: enumeration value 'CLIMATE_SWING_BOTH' not handled in switch [-Wswitch]
164 | switch (swingMode)
| ^
src/gree_ir.h:164:14: warning: enumeration value 'CLIMATE_SWING_HORIZONTAL' not handled in switch [-Wswitch]

But compilation still happens and works, but not entirely correctly.

@Hagakurje
Copy link
Owner

It is just warning, not error. If your AC supports that features, you can add code for them in gree_ir.h.

@Oleg-Sob
Copy link

Part of the code doesn't work

ac.setTemp(22);

When rebooting, it sets the temperature to 0, but it needs 22. Therefore, the temperature regulator is not visible.

@Hagakurje
Copy link
Owner

When rebooting, it sets the temperature to 0, but it needs 22.

Create automation to restore temp after reboot.

Therefore, the temperature regulator is not visible.

I don't know, why regulator is not visible. HA with last updates + EspHome with last updates + default thermostat card - regulator is visible.

@kevindebosscher
Copy link
Author

I just found a work-around by starting a little script as HA boots. After the script has been executed; I can set temps
IMG_3655

@Oleg-Sob
Copy link

You can do this, but it would be good if it worked without a script.

@hldh214
Copy link

hldh214 commented Mar 18, 2024

My solution is to manually set target_temperature in setup(), like this:

  void setup() override
  {
    ac.begin();
    ac.setMode(kFujitsuAcModeCool);
    ac.setTemp(22);
    ac.setFanSpeed(kFujitsuAcFanAuto);
    ac.setSwing(kFujitsuAcSwingBoth);
    ac.off();

    this->mode = CLIMATE_MODE_OFF;
    this->fan_mode = CLIMATE_FAN_AUTO;
    this->swing_mode = CLIMATE_SWING_BOTH;
    this->target_temperature = 22;  // <---- here
    this->publish_state();
  }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants