-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Add new device: IKEA SOMRIG E2213 #6574
Conversation
Can you provide the debug logging when these actions are sent? See https://www.zigbee2mqtt.io/guide/usage/debug.html on how to enable debug logging. |
I observe the following pattern: |
Could you also provide the herdsman debug logging of this? One click is enough. See https://www.zigbee2mqtt.io/guide/usage/debug.html on how to enable the herdsman debug logging. Note that this is only logged to STDOUT and not to log files. |
when 1 pressed short:
|
Thanks, I've added the cluster now, with this you will now get the parsed messages instead of Changes will be available in the dev branch in a few hours from now. (https://www.zigbee2mqtt.io/advanced/more/switch-to-dev-branch.html) |
After some digging I found this issue: Koenkk/zigbee2mqtt#16808 which discussed the implementation of the dot buttons of the symfonsik controller lengthily.
Besides, the new cluster broke the dot buttons for the Symfonisk remote:
|
…erters#6574" This reverts commit 6e454d8.
based on symfonisk remote
This will not work anymore with z2m 1.34.0, please test again that version. |
Yes, thats why I added the pr on herdsman itself. |
Can you try with this converter: https://gist.github.com/Koenkk/ad301d469171bfe1b977a4e021e821cc#file-ext-js-L33 ? |
The converter throws the exception for the symfonisk as described in Koenkk/zigbee2mqtt#20003 |
Pushed a fix for this, please check again with latest dev Changes will be available in the dev branch in a few hours from now. |
Looks like this device is using Matter ZCL like Symfonisk gen 2 on manufacture cluster (Likely IKEA is updating the firmware using matter cluster in near future on both devices then HA can taking care of the commands) with production firmware (not factory) and only need EP 1 and 2 being bonded then looking in sniffs from it so shall being easy getting initial, single double long and long release working. |
src/devices/ikea.ts
Outdated
@@ -281,10 +281,26 @@ const fzLocal = { | |||
} satisfies Fz.Converter, | |||
ikea_dots_click_v2: { | |||
// For remotes with firmware 1.0.32 (20221219) | |||
cluster: 'heimanSpecificScenes', |
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 this doesn't work anymore in the latest dev branch
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.
Me as well. Must be an error from fetching upstream.
src/devices/ikea.ts
Outdated
@@ -281,10 +281,26 @@ const fzLocal = { | |||
} satisfies Fz.Converter, | |||
ikea_dots_click_v2: { | |||
// For remotes with firmware 1.0.32 (20221219) | |||
cluster: 'heimanSpecificScenes', |
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.
Try with:
cluster: 'tradfriButton',
type: ['commandAction1', 'commandAction2', 'commandAction3', 'commandAction4', 'commandAction6'],
should be ready now |
src/devices/ikea.ts
Outdated
commandAction6: 'double_press', | ||
}; | ||
const action = utils.getFromLookup(msg.type, lookup); | ||
return {action: `dots_${button}_${action}`}; |
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 we can remove the dots_
prefix here (since it's always there). After that this can be merged.
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.
Good point!
Thanks! |
Is there a working externernal converter at the moment so I can bridge the time until there is new Home Assistant addon edge release? Thank you in advance |
@kafisc1 ca be found here |
Hi, Hopefully you can help? Using HS4 & ZIGBEE Plus, and this external controller . Also, the exposes in the converter "e.battery().withAccess(ea.STATE_GET), e.action(['dots_1_initial_press', 'dots_2_initial_press', 'dots_1_long_press', 'dots_2_long_press', 'dots_1_short_release', 'dots_2_short_release', 'dots_1_long_release']), ]," is it missing 'dots_2_long_release' ? Sorry if I'm in the wrong place, |
I was able to snatch a few of the yet (to my knowledge) unreleased SOMRIG Shortcut buttons and got it successfully configured.
The button actions are transferred by
heimanSpecificScenes
. Not sure how the handle them for now.Advice how to use them would be greatly appreciated.