basbruss / adaptive-cover

An Adaptive Cover component for HomeAsisstant to control covers based on the sun's position
MIT License
90 stars 11 forks source link

Temperature is in between min and max, but blinds are fully closed #203

Closed Dielee closed 3 months ago

Dielee commented 3 months ago

What version of Adaptive Cover are you using?

v1.2.2

What version of Home Assistant are you using?

2024.6.0

Checklist

Describe the issue

Hello! first, thanks for your effort and this integration.

The temperature from the sensor entity (21,4) is between min (18) and max (23,5). So, what the flow looks like, the blind should be opend, but they are closed. Not 0%, but 4%, 6% and so on...I don't understand why the shutters are closed. Maybe this isn't a bug, just an understanding question.

2024-06-07 08_45_03-Einstellungen – Home Assistant 2024-06-07 08_45_08-Einstellungen – Home Assistant 2024-06-07 08_45_30-Einstellungen – Home Assistant 2024-06-07 08_46_19-Einstellungen – Home Assistant

Reproduction steps

  1. Activate climate mode
  2. Input sensor entity for inside temp
  3. Set min and max in between the inside temp
  4. Covers should be open

Diagnostics dump

No response

basbruss commented 3 months ago

So the cover is not adjusted to the calculated position of the sensor? It can be the end_time, minimum delta between position changes.

See also this issue #202

Dielee commented 3 months ago

No, this works. But shouldn't stay the cover open in between the min and max temp ?

basbruss commented 3 months ago

Yes, when presence is detected or no presence sensor is set, our only goal in the intermediate mode is to reduce glare while letting in valuable natural light. This holds for all three temperature modes with presence. Fully closed is never the wanted strategy if the sun is in front of the window and there is presence because this integration is built around the principle that we want as much natural light as possible while reducing glare and direct light on our workspaces/into our rooms

Dielee commented 3 months ago

There is no presence set at this time. At this time the room is dark and temperature is in between min and max. So what can i do ?

basbruss commented 3 months ago

Just to understand things better. Do you have shutters like Rollläden that do not directly open? Does the position of the sensor correspond with the position of the cover entity?

There is no presence set at this time

What do you mean by this?

  1. no presence entity is configured
  2. Presence is not detected
Dielee commented 3 months ago

I have shutters which are closed, if the room temperature is in between min an max temp. The position of the sensor correspond with the position, yes.

  1. It is no presence entity configured
basbruss commented 3 months ago

Does it work when you toggle the climate mode off?

Dielee commented 3 months ago

I can't test now, as the sun isn't in front of my window anymore. So there are at 100% now...

basbruss commented 3 months ago

Okay, I think if have reproduced a similar looking issue. Can it be that your default position is 0%? Because I do get the default position in that situation where actually the calculated position is expected.

Dielee commented 3 months ago

No, default is 100%. Maybe i'm able to reproduce with another group of adaptive covers. Should i try?

basbruss commented 3 months ago

Yes, please also share as many of your settings, so I can reproduce it. (future releases will enable diagnostics download from HA to include all the settings)

Dielee commented 3 months ago

Okay, i will try! Give me some time

Dielee commented 3 months ago

I rebuild this issue for my south windows and it's the same. Here are my settings:

2024-06-07 13_24_03-Übersicht – Home Assistant 2024-06-07 13_24_59-Einstellungen – Home Assistant 2024-06-07 13_25_12-Einstellungen – Home Assistant 2024-06-07 13_25_21-Einstellungen – Home Assistant 2024-06-07 13_25_36-Einstellungen – Home Assistant 2024-06-07 13_25_49-Einstellungen – Home Assistant 2024-06-07 13_26_09-Entwicklerwerkzeuge – Home Assistant

Dielee commented 3 months ago

Current service state:

2024-06-07 13_28_46-Einstellungen – Home Assistant

basbruss commented 3 months ago

Okay, this clears some things up for me. So first of all the change in the position of the sensor must be at least 10% higher/lower than the cover currently is. So cover at 0% and sensor to 10% results in cover to 10%, but cover at 0% and sensor to 9% results in cover to 0%.

Secondly, the distance (shaded area) is 0.1m to the workspace from the cover. This is very low and translates to that we only let in light if the elevation angle is very steep, hence the calculated position is very low. Which corresponds to the actual sensor state. The lower the elevation angle of the sun, the lower the cover must be to block out direct light on a workspace surface 10 centimetres from the window.

So I don't think this is a bug for now, but rather a misunderstanding of the mechanics and algorithm used.

You could increase the distance (shaded area) since the height of the surface plane is equal to the bottom of the window, so 0.5m will in most cases not produce glare to your eyes.

Dielee commented 3 months ago

Ah alright! But i don't understand shaded area, maybe i am just dump ?.... I changed to 0,5 and it works now. Thanks!

Dielee commented 3 months ago

But, are the covers normaly not fully openend when temperature is in between min and max ?

basbruss commented 3 months ago

But, are the covers normaly not fully openend when temperature is in between min and max ?

No, since this integration aims to reduce visible glare to occupants while maximizing natural light coming in. Fully open with presence will lead to disturbing glare. If you want to understand it better I highly recommend reading this paper, although it is aimed to tilted blinds, the same principles hold for covers in general, be it that the calculation of the angles is slightly different.

Dielee commented 3 months ago

Ahhh, great! This makes sence. Thanks for your patience!