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

Stop function #2

Open
gbeleris opened this issue Jan 10, 2024 · 11 comments
Open

Stop function #2

gbeleris opened this issue Jan 10, 2024 · 11 comments
Assignees
Labels
enhancement New feature or request

Comments

@gbeleris
Copy link

Many thanks for this. Any chance you could add a stop function? I have a 3rd switch that acts similar to the stop button on the remote.
Thanks?

@Sese-Schneider Sese-Schneider added the enhancement New feature or request label Jan 11, 2024
@haigaru
Copy link

haigaru commented Jan 11, 2024

I also have the same problem: stop button

@Sese-Schneider
Copy link
Owner

@gbeleris @haigaru I have a few questions before adding this function.

Right now I'm wondering how your implementation is done (please share your configuration).
For the open_switch_entity_id and close_switch_entity_id are you using a switch or a button? (I don't think my integration would work with a button, but please correct me if I'm wrong)

What should be implemented?
A stop_button_entitiy_id or a stop_switch_entitiy_id? How would a switch to stop the cover even work?

Also, if the entity you're using is indeed a button, couldn't you just create a automation which would toggle the open & close switch off every time the button is pressed?

@Sese-Schneider Sese-Schneider added the question Further information is requested label Jan 12, 2024
@haigaru
Copy link

haigaru commented Jan 22, 2024

I am using the old integration "Cover Time Based RF (script/entity)"

cover configuration

  • platform: cover_time_based
    devices:
    room_rolling_shutter:
    name: Room Rolling Shutter
    open_switch_entity_id: script.workroom_blinds_3_4_open
    close_switch_entity_id: script.workroom_blinds_3_4_close
    traveling_time_down: 47
    traveling_time_up: 47
    tilting_time_down: 2.3
    tilting_time_up: 2.7

@gbeleris
Copy link
Author

I have 3 relays connected to the cover, up, stop, down. They are interlocked, when one turns on the others go off.
if there was a stop_switch_entitiy_id I would assign the relay switch I have for the stop function. Hope it's workable. Thanks!

@voidspark99
Copy link

+1; I have RF controlled blinds, with up, down and stop buttons.

@fvatter
Copy link

fvatter commented Feb 16, 2024

@voidspark99 is it working well for you?

@haigaru
Copy link

haigaru commented Mar 14, 2024

@gbeleris @haigaru I have a few questions before adding this function.

Right now I'm wondering how your implementation is done (please share your configuration). For the open_switch_entity_id and close_switch_entity_id are you using a switch or a button? (I don't think my integration would work with a button, but please correct me if I'm wrong)

What should be implemented? A stop_button_entitiy_id or a stop_switch_entitiy_id? How would a switch to stop the cover even work?

Also, if the entity you're using is indeed a button, couldn't you just create a automation which would toggle the open & close switch off every time the button is pressed?

It's actually very difficult to express because I'm quite poor at English. I use Google Translate to translate my language back into English.

My problem (and someone else's here)

Conclusion: we really want the author to add a stop option. Because currently, using your integration, I can only open and close the curtain. The option to adjust the tilt of the curtain does not work because it cannot stop the curtain.

@Sese-Schneider
Copy link
Owner

Sese-Schneider commented Mar 14, 2024

I've added an optional stop_switch_entity_id for stopping the cover.
It supports a switch which will toggle on whenever the cover needs to be stopped.

It is published as a pre-release here: https://github.com/Sese-Schneider/ha-cover-time-based/releases/tag/v2.1.0-beta.1
Select Beta Version when redownloading the repository. Tutorial

BwotbU3quwhyyzPE

@haigaru @gbeleris @voidspark99 @fvatter
Please let me know if this solves your problems, then I will publish it as a normal release.

@Sese-Schneider Sese-Schneider self-assigned this Mar 14, 2024
@Sese-Schneider Sese-Schneider removed the question Further information is requested label Mar 14, 2024
@gbeleris
Copy link
Author

thanks for all your efforts with this!

The entity comes up as unavailable after adding the yaml code for it!
Any thoughts why this is happening ?

Screenshot 2024-03-14 222031
Screenshot 2024-03-14 222023

@Sese-Schneider
Copy link
Owner

Sese-Schneider commented Mar 15, 2024

@gbeleris you've found a bug!
The issue was the integration would not load when the tilting entities were not present

Its fixed in the newest pre-release https://github.com/Sese-Schneider/ha-cover-time-based/releases/tag/v2.1.0-beta.2

@haigaru
Copy link

haigaru commented Mar 17, 2024

I've added an optional stop_switch_entity_id for stopping the cover. It supports a switch which will toggle on whenever the cover needs to be stopped.

It is published as a pre-release here: https://github.com/Sese-Schneider/ha-cover-time-based/releases/tag/v2.1.0-beta.1 Select Beta Version when redownloading the repository. Tutorial

BwotbU3quwhyyzPE BwotbU3quwhyyzPE

@haigaru @gbeleris @voidspark99 @fvatter Please let me know if this solves your problems, then I will publish it as a normal release.

Thank you very much, it works perfectly. But there are some small problems as follows:

  • This curtain when tilting down and tilting up will have three states:
  • Status 1: cover all the way up
    z5258749230332_2bd67ea8a6240624c23fdc1ad97c499f
  • Status 2: cover all the way down
    z5258749229931_b142e248b740726b2b25f0421238bb81
  • state 3: cover all the parallel
    z5258749160443_0731a038d18da78cdcae1aebb1fb8217

Currently the tilting down and tilting up adjustment toolbar can only adjust states 2 and 3.

In my opinion, you should add the tilting down and tilting up bar: 0% is state 2, 50% is state 3, 100% is state 1
Screenshot 2024-03-17 at 22 08 10

Once again I thank you very much (I'm sorry for the way I use English)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants