thomasgermain / vaillant-component

Multimatic integration for Home Assistant (also compatible with sensoAPP)
MIT License
137 stars 24 forks source link

How to set duration for quick veto? #191

Closed hvbargen closed 1 year ago

hvbargen commented 1 year ago

Note: I am using VR921 and Xavier Griffons PR.

If I rotate the temperature wheel in the HA UI, this works: I can see in the SensoComfort that the target temperature is changed per quick veto for 6 hours.

How can I configure this duration?

BTW If I try to call the service set_quick_veto manually, this results in an error message:

Unexpected error for call_service at pos 1: Cannot put https://smart.vaillant.com/mobile/api/v4/facilities/.../systemcontrol/tli/v1/zones/Control_ZO1/configuration/quick_veto, status: 400, response: {"errorCode":"INVALID_PARAMS"},payload: {'temperature_setpoint': 22.0, 'duration': 60.0} image

hvbargen commented 1 year ago

@xaviergriffon Regarding the how-to-set question: It seems that the 6 hours is hard-wired in coordinator.py line 254:

veto = QuickVeto(6, target_temp)

Can we make this configurable somehow?

xaviergriffon commented 1 year ago

Good test! There is a unit conflict between Multimatic and Senso. For Senso, the unit is in hours. We will have to adapt the controls according to the application.

For the default duration parameter, I think it should be set in the configuration as for the number of minutes between refreshes.

hvbargen commented 1 year ago

Yeah, that sounds reasonable.

xaviergriffon commented 1 year ago

I pushed a correction on my branch. You need to re-download the addon and after restarting, check the commit.

Capture d'écran 05 02 2023 à 19 20 55 PM Capture d'écran 05 02 2023 à 19 22 51 PM
thomasgermain commented 1 year ago

In multimatic:

In senso:

hvbargen commented 1 year ago

On my SensoComfort, the default duration is 3h.