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

Features/add support open and close time #129

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

andrea-parisi
Copy link

I have provided two new attributes for cover objects to determine their status (only if they are not advanced).
The attributes are optional.

Exemple:

cover:
    shutter_11:
      where: '11'
      name: Cover 11
      advanced: False
      manufacturer: BTicino
      opening_time: 27
      closing_time: 27

@nicolacavallo84
Copy link

your work is interesting

@michnovka
Copy link

This is great! I especially appreciate that you set different times for open and close - this is something that was bothering me in openhab configuration, I didnt get why the % get out of sync so fast. And of course - physics :D It goes down fastet than up.

Can we have this merged please?

@marcob79
Copy link

marcob79 commented Dec 8, 2024

this mod is very interesting.
Up to now I was using, on top of @anotherjulien myhome integration, the following
https://github.com/nagyrobi/home-assistant-custom-components-cover-rf-time-based

so, I have a cover template that handle time position that leverage on standard myhome without position

I quickly tried your modification @andrea-parisi and looks what I was looking for in order to have everything combined.
The only difference that I observed, is that nagyrobi time based cover, updates the position of the cover in real time while it is opening or closing, while this modification, if I saw correctly, updates the cover position when it stops.
Can you consider to further improve this feature updating cover position in real time?
thanks
Marco

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

Successfully merging this pull request may close these issues.

4 participants