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

Automation turning on Climate React changes Climate React configuration #148

Open
mkrexp opened this issue Jul 29, 2024 · 11 comments
Open

Comments

@mkrexp
Copy link

mkrexp commented Jul 29, 2024

My automation is configured to automatically turn on climate react at 10:00 PM. When automation is turning on Climate React , Climate React setup is changed automatically ('AC Light' and 'Swing').
My configuration before automation runs is shown below:
before

After automation runs configuration is changed as presented below:
after

@benwebbbenwebb
Copy link
Collaborator

Hi @mkrexp

What automation are you running? Is it as simple as turning on the ClimateReact switch?

Do you have the (new) ClimateReact Auto Setup turned on?

If you are only switching ClimateReact on (or off) via Home app automation then I would expect it to not change the settings configured in the Sensibo app as you expect, however the new Auto Setup mode does “copy” your current settings.

@mkrexp
Copy link
Author

mkrexp commented Jul 30, 2024 via email

@benwebbbenwebb
Copy link
Collaborator

Thanks for the reply @mkrexp

Do you have the Light switch enabled in your settings for this plugin? By the feel of it I think 'defaults' are being used by Sensibo... so if Light isn't set/sent by Home app / Homebridge Sensibo must default it to On.

Similarly for Swing, Apple Home app doesn't support modes other than On / Off, so your setting of 'Fixed top' is being overwritten by the default of 'Stopped (auto)'.

Please let me know your thoughts on this, will help direct further investigations (and any potential fix/changes).

@mkrexp
Copy link
Author

mkrexp commented Jul 30, 2024

Thanks for the reply @benwebbbenwebb

I have the light switch enabled in plugin config. I have asked Sensibo Support if they change anything in climate react settings and they do not change anything.

Before updating from 2.3.4 to 2.5.1, configuration was not overwritten by described automation or when I turned climate react from home app, so I thought something is wrong with plugin, not Sensibo.

@benwebbbenwebb
Copy link
Collaborator

Thanks @mkrexp

Agree that it sounds logical for it to be the plugin, I’ll take another look at the code changes.

Would you be able to toggle the light switch a few times, and then change it to your desired setting for ClimateReact (I think Off?) and see if turning on ClimateReact via Home still changes the desired Light setting?

With the Swing mode, that one is going to be harder… Apple don’t support fixed swing positions (like Fixed top) so we don’t expose it. Will need to think about how best to achieve your desired outcome, but still allow others to setup and modify the ClimateReact settings via Home app (rather than having to use Sensibo app).

@mkrexp
Copy link
Author

mkrexp commented Aug 4, 2024

Hi @benwebbbenwebb

Toggling light switch works as turning on/off AC but without turning it on or off, rather like 'settings switcher' - light is turned on/off but swing is also changed. For example, I have AC turned on with light on and swing set to fixed top. After toggle light switch in Home app, on my AC I have light switched to off and swing is changed to 'stopped (auto)'. In logs available in Sensibo app I can see entry 'API call API call(Michal) set the swing to stopped (auto)'. After next toggle of light switch in Sensibo app logs I can see entry 'API call API call API call(Michal) kept state on'.

Maybe you should send only dedicated commands (i.e. 'turn climate react on') without any other parameters? I don't know if Sensibo API allows it, but I suppose that was done in version I used before updating to 2.5.1.

It's very frustrating that I can configure automation for turning on features (like climate react) but I have to remember change climate react settings to my preferences simultaneously

@benwebbbenwebb
Copy link
Collaborator

Thanks again @mkrexp for the added info

Hi @seidnerj
Looks like some of the changes to ClimateReact and the API calls in the currently version are having negative side effects for some users.

Specifically sending the whole ClimateReact object when enabling (or disabling). As Homebridge/Home app don’t support multiple swing positions the new implementation is overwriting the swing (and light?) setting…

Would appreciate your thoughts on the best way to proceed @seidnerj.

@seidnerj
Copy link
Contributor

seidnerj commented Aug 9, 2024

Sure, I'll fix that. Will update.

@seidnerj
Copy link
Contributor

Hey guys, I'm not sure when I'll get to this, I'm swamped at the moment, will work on this when I can. Sorry!

@benwebbbenwebb
Copy link
Collaborator

All good @seidnerj

I might try and take a look, see what can be done in the current version…

@mkrexp in the interim, to get your previous process working again, have you tried downgrading back to the v2.3.4 version?

@seidnerj
Copy link
Contributor

@benwebbbenwebb 🙏

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

3 participants