-
Notifications
You must be signed in to change notification settings - Fork 4k
Fixed fls-m from dresden-elektronik #11078
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
| const e = exposes.presets; | ||
|
|
||
| const tzLocal = { | ||
| flsm_color_hs: { |
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.
Why not use
| export const light_color: Tz.Converter = { |
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.
Because when using tz.light_color the device did not change color. I had to build this custom solution for the device to accept color changes.
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.
I think because it doesn't support x/y, can you try with just h/s: m.light({endpointNames: ["l1", "l2", "l3", "l4", "l5"], color: {modes: ["hs"]}})
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.
The challenge with this product is that the endpoints are dynamic. The device has DIP switches that configure which endpoint is exposed. As far as I understand, the extend field and the m.light function are static, so I had to work around this using the exposes and configure fields. In the end, this worked perfectly and corrected the previous “simple” implementation, which was incomplete (it only exposed five static lights, but the actual behavior depends on the DIP switch configuration).
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.
Maybe there is a different way to implement this? I am new to zigbee2MQTT.
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.
Yes we can identify using fingerprint (so a different definition is used for every mode): example:
| fingerprint: [ |
No description provided.