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

fix(lumi): pet feeder LED indicator control #8041

Merged

Conversation

impankratov
Copy link
Contributor

@impankratov impankratov commented Sep 28, 2024

Basically a revert of 3a5c6ae

After updating device to firmware version 9761, LED control doesn't work at all. Zigbee2mqtt reports "OFF" state, while indicator is turned on. Controlling the LED state either via zigbee2mqtt or via Home Assistant doesn't result in any change.

I can only assume that previous firmware version (9505) got incorrect status mapping, while latest (9761) fixed this.

Update:

Did some research and turns out, device logic is not so simple: led_indicator controls LED indicator night mode (with hard-coded time period). Added some description for this property.

Tested via external converters, works great!

refs:

After updating to firmware version 9761, LED control doesn't work at
all. Zigbee2mqtt reports "OFF" state, while indicator is turned on.
Controlling the LED state either via zigbee2mqtt or via Home Assistant
doesn't result in any change.

I can only assume that previous firmware version (9505) got incorrect
status mapping, while latest (9761) fixed this.
@impankratov
Copy link
Contributor Author

This is how LED control behaves in Home Assistant:
https://github.com/user-attachments/assets/ce4a4981-2098-4150-9840-dee034816016

@impankratov impankratov marked this pull request as draft September 28, 2024 11:53
@Koenkk
Copy link
Owner

Koenkk commented Sep 28, 2024

LGTM! Let me know if it can be merged.

@impankratov
Copy link
Contributor Author

@Koenkk sorry for bothering, tried this fix with external converter and it doesn't work - LED stays "on" no matter what.
Will re-open this if find any solution.

@impankratov impankratov deleted the bugfix/aqara-pet-feeder-led-indicator-control branch September 28, 2024 13:28
@impankratov impankratov reopened this Sep 28, 2024
@impankratov impankratov marked this pull request as ready for review September 28, 2024 17:06
@Koenkk
Copy link
Owner

Koenkk commented Sep 29, 2024

Ready for merge?

@impankratov
Copy link
Contributor Author

@Koenkk Yep!

@Koenkk Koenkk merged commit e7bd227 into Koenkk:master Sep 29, 2024
4 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Sep 29, 2024

Thanks!

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.

2 participants