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

On "stop" from external remote MQTT only reports the target position, but not current position #518

Open
smumriak opened this issue Jan 1, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@smumriak
Copy link

smumriak commented Jan 1, 2025

Hardware

ESP32

Firmware version

v2.4.7

Application version

v2.4.7

What happened? What did you expect to happen?

Inverted positioning! Sending shades down from external remote. Shade starts moving, it's target is set to 0, position gets updated. Sending shades stop from external remote. Last position update is at 89%, target position update is at 88%. This unfortunately breaks integration with homebridge, shades are in permanent "closing state" after stopping movement.

How to reproduce it (step by step)

1. Press down on external remote.
2. Press stop on external remote.
3. Observe MQTT logs.

Logs

espsomfy/shades/2/target 0
espsomfy/shades/2/cmdSource remote
espsomfy/shades/2/cmdAddress 15315477
espsomfy/shades/2/cmd Down
espsomfy/shades/2/direction 1
espsomfy/shades/2/position 99
espsomfy/shades/2/position 98
espsomfy/shades/2/position 97
espsomfy/shades/2/position 96
espsomfy/shades/2/position 95
espsomfy/shades/2/position 94
espsomfy/shades/2/position 93
espsomfy/shades/2/position 92
espsomfy/shades/2/position 91
espsomfy/shades/2/position 90
espsomfy/shades/2/position 89
espsomfy/shades/2/target 88
espsomfy/shades/2/cmdSource remote
espsomfy/shades/2/cmdAddress 15315477
espsomfy/shades/2/cmd My
espsomfy/shades/2/direction 0
@smumriak smumriak added the bug Something isn't working label Jan 1, 2025
@smumriak
Copy link
Author

smumriak commented Jan 1, 2025

I expect that upon stopping the code would publish both position and then target at 88 in this example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant