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

No trigger to move cover #180

Closed swa72 closed 4 months ago

swa72 commented 4 months ago

What version of Adaptive Cover are you using?

1.2.0

What version of Home Assistant are you using?

2024.5.5

Checklist

Describe the issue

I have set an entity to determine start time (Start Time Entity). I would assume that at this point in time the automation sets the cover to its default position of 85% (Default Position, Initial position of the cover in the absence of sunlight glare detection). In my case, the cover did not move.

image

Start time entity is a helper with

has_date: false
has_time: true
editable: true
hour: 8
minute: 0
second: 0
timestamp: 28800
friendly_name: cover_result

Reproduction steps

1. 2. 3. ...

Diagnostics dump

No response

basbruss commented 4 months ago

The time is not a trigger for that time. It only prevents adaptive changes before that time. So it starts changing after that time, but not directly on the time itself.

swa72 commented 4 months ago

Ok, misinterpreted the docs ;-) You may want to add something to the start time description.

basbruss commented 4 months ago

Always happy to accept improvements by a Pull request :wink: