-
Notifications
You must be signed in to change notification settings - Fork 7
add tilt/angle #8
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
base: master
Are you sure you want to change the base?
Conversation
added usage for tilt position
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Roland,
thanks for the pull request. Looks very good overall.
I left a couple of comments with suggestions on how I would change things. Have a look.
My one big concern (it's also in the comments) is if position and tilt should be send as seperate commands. Have you checked if this is how it is done be the webcontrol server? Can they not be send in one command, e.g., '03{position}{tilt}ffff' ?
Best
Cornelius
Co-authored-by: Cornelius Mund <4417017+cornim@users.noreply.github.com>
Co-authored-by: Cornelius Mund <4417017+cornim@users.noreply.github.com>
Co-authored-by: Cornelius Mund <4417017+cornim@users.noreply.github.com>
Co-authored-by: Cornelius Mund <4417017+cornim@users.noreply.github.com>
Co-authored-by: Cornelius Mund <4417017+cornim@users.noreply.github.com>
Co-authored-by: Cornelius Mund <4417017+cornim@users.noreply.github.com>
Co-authored-by: Cornelius Mund <4417017+cornim@users.noreply.github.com>
While looking through this PR I realized that there is one thing missing. So far only one type of blind was supported so it wasn't necessary to distinguish different product types but if we want to integrate this properly into HA, the library needs to carry the information which product type a shade has. |
will check this hopefully tonight |
sorry for the late response, for explanation, I'll provide exact model types later, for case it helps someone if i can do any further check or investigation, pls let me know regarding the move/tilt in one command, id dint have time yet, hopefully tonight |
Thanks for looking it up. It seems that between the model that you have and the model that I have we can only distinguish between with/without tilt and we can use bedientyp for it. |
unfortunately it's the same for me too (time management is very challenging with one-year-old twins, haha) Regarding your HA integration, I tried to implement the stop function, added SUPPORT_STOP in the import and feature section and unfortunately I always get a Shade Object has no Attribute stop_moving_shade error. do you have any idea what i missed here? |
Hi Cornim,
as discussed earlier that day, here is my pull request.
tilt can be adjusted from 30 (open) and 100 (fully closed)
shades[1].set_shade_tilt_position(100)
with shades[1].get_shade_state(True) every of my 6 raffstores (venetian blinds) shows slightly different tilt values, so this are the "lowest common denominator", and the difference in tilt/ angle is imperceptible.
shades[1].stop_moving_shade() stops surprisingly a moving shade, sometimes it needs some seconds bust most of my trials it stops during 1or 2 seconds
please dont hesitate to reject if the quality is insufficient, my coding skills are very limited (as you have surely noticed).
if you have any pointers or clues regarding the HA Integration, it would be really helpful
cheers
roland