Closed h3rbst closed 8 months ago
Hey there @julius2342, @farmio, @marvin-w, mind taking a look at this issue as it has been labeled with an integration (knx
) you are listed as a code owner for? Thanks!
(message by CodeOwnersMention)
knx documentation knx source (message by IssueLinks)
Hi 👋! See https://github.com/home-assistant/core/issues/96129 for the reason behind the change.
Unfortunately the UIs default cover card just calls the close_cover_tilt
method. It is not expected to do stepwise movements. I guess the intended way is to hit the "stop" button.
There are 3 workarounds coming to my mind currently
angle_address
from the config. This will however have tradeoffs - I don't think the tilt-slider would work properly anymore then. I'm very sorry but totally not with you. I think in the past it worked 100% as it should, because it worked like normal KNX-Button/Panel. There you can move the covers, in my case raffstores, a) completely up or down with long press b) move them step by step so that the lamellae turn around and you can try out which angle fits best
For me it makes absolutely no sense to have two buttons where I can "differentiate" between a) Lamellae are completely closed and no sun comes through b) Lamellae are completely (but the other way around) closed and no sun comes through → same result as a)
Are you using Raffstores as well? Can not imagine you want it in another way :-P
The workarounds for me does not make sense since:
Would really love if we could find a way so it worked like before and to be KNX STANDARD again
@farmio how could we go on here to fit both worlds needs?
Unfortunately it is not possible. The UI element calls the cover.close_cover_tilt
and the service expects the cover tilt to close.
There is no "close cover stepwise" service in HA. Imho it was implemented wrong before.
Feel free to get a second opinion about that - I don't even have tiltable covers, so no idea how other integrations manage those things.
Ad 1: you can use knx.send
to send the telegram you like.
@farmio So why does the KNX integration offers the possibility to maintain the move_short_address when it is not possible to use it?
Edit: And the documentation says move_short_address: "KNX group address for moving the cover stepwise up or down. "
@marvin-w would love to hear you opinion as well if possible? Do you have raffstores or just normal roller shutters?
To stop the cover. Or to move covers that don't support angle positions.
@farmio Home Assistant KNX documentation says move_short_address: "KNX group address for moving the cover stepwise up or down. "
@h3rbst yes that is correct. It's the address for moving step wise. The problem is that HA architecture have no notion/api to perform that command. The buttons in UI that was used before was not intended for that purpose. KNX integration need to adhere to how home assistant architecture is defined.
I do agree stepwise move would make sense in both the tilt and position features. But it needs to be fixed at another level first.
There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.
The problem
Hi, I updated to the newest Version 2023.8 of Home Assistant. I do not understand the reason for Pullrequest #96192 , but theses changes for me break my KNX covers. Before the update I was able to tilt my covers step by step in the UI. Now this is not possible anymore, since it does not use the move_short address anymore and just sends the angle position 0 e.g. Please revert this pullrequest or give an option to use the move_short address again.
What version of Home Assistant Core has the issue?
core-2023.8.1
What was the last working version of Home Assistant Core?
core-2023.7.3
What type of installation are you running?
Home Assistant OS
Integration causing the issue
KNX
Link to integration documentation on our website
https://www.home-assistant.io/integrations/knx/
Diagnostics information
No response
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response