jmcollin78 / versatile_thermostat

A full featured Thermostat for Home Assistant: presets, window, motion, presence and overpowering management
MIT License
328 stars 34 forks source link

[5.0.0] - Unable to set "window open" settings values #280

Closed MaxWaldorf closed 11 months ago

MaxWaldorf commented 11 months ago

The windows open detection windows to enter values, keeps on coming up and doesn't validate...

WarC0zes commented 11 months ago

Hi, Yes, I've this error in the log. We cannot finalize a configuration.

2023-12-18 08:02:24.683 ERROR (MainThread) [custom_components.versatile_thermostat.config_flow] Only one window detection method should be used. Use window_sensor or auto window open detection but not both
jmcollin78 commented 11 months ago

Hello, hum I see the problem.

You should uncheck the "Use central configuration" for the window and remove the values for auto window detection. I will improve this but it should be working with that

WarC0zes commented 11 months ago

Hello, hum I see the problem.

You should uncheck the "Use central configuration" for the window and remove the values for auto window detection. I will improve this but it should be working with that

if we delete the values of the auto window detection and validate. This resets the default values and we cannot continue to finalize the configuration.

jmcollin78 commented 11 months ago

I wonder why I don't run into this case. I will certainly not be able to fix today.

jmcollin78 commented 11 months ago

I try on my side.

On the "Central configuration" I've got this:

Capture d’écran 2023-12-18 à 10 14 15

In the VTherm configuration itself, I've got this: Capture d’écran 2023-12-18 à 10 15 16

And this is working fine.

If I uncheck the "Use central configuration" :

Capture d’écran 2023-12-18 à 10 16 35

I cannot validate this window because the both window mode are configured. Capture d’écran 2023-12-18 à 10 17 40

The error is not displayed and that is the bug for me.

And you are right, when I remove the auto detection open window parameters: Capture d’écran 2023-12-18 à 10 18 51

The error is still there (but not displayed) and that is the same bug.

Workaround

So the workaround is to add a window sensor in the previous page and check "Use central configuration" or to deactivate "With window detection" in the first configuration page.

Capture d’écran 2023-12-18 à 10 21 12

WarC0zes commented 11 months ago

I used the central configuration and I was able to finalize the Vtherm configuration. But you have seen that if we don't use the central configuration for the openings, we cannot finalize the configuration.

jmcollin78 commented 11 months ago

https://github.com/jmcollin78/versatile_thermostat/releases/tag/5.0.1