-
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 NHMOTION/UNIDIM/1 (ELKO EKO06984, EKO06985 and EKO06986) #8187
Conversation
Just a quick note that I think the vendor should probably be Schneider. The ELKO version is just a white-label of WDE002962. Also, I have a list of commands sniffed from the gateway during pairing, and when executing commands in the app. Might help with getting all the functionality implemented.
|
src/devices/elko.ts
Outdated
model: 'EKO06984', | ||
vendor: 'ELKO', | ||
description: 'SmartPir with push dimmer', | ||
fromZigbee: [fz.on_off, fz.brightness, fz.illuminance, fz.level_config, fz.schneider_lighting_ballast_configuration, fz.occupancy], |
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.
Can you try with modern extend? E.g. extend: [light({configureReporting: true})],
, then you can remove fromZigbee
, toZigbee
, exposes
and configure
.
From your list and some testing it seems like some custom tz an fz might be required to support all features. For example, setting
I'll try to have a go this weekend, but all suggestions are welcome :) |
Apologies, on closer inspection I've realized my switch is a slightly different version from yours. I have the For what it's worth I'm seeing the same errors when setting dimmer_mode 1 or 2. In the past I've used the |
I see. I think EKO07252 is the replacement for EKO06984/85/86, as those are not sold anymore. I'll change the vendor back to Elko then I think, since I haven't been able to find any alternative producers for this older dimmer. I'll try to do some experimentation regarding dimmer mode. |
Some updates:
Here's the endpoints and clusters
|
That'll be from the device. You can query it in the dev console by reading Some updates on my end.
I'm planning to submit a PR for my device since it's slightly different. We should be able to share most of the implementation. BTW, there's some Schneider documentation floating around for dimmer switches. It doesn't include PIR functionality, but it's hugely useful for getting accurate documentation of dimmer functionality. |
Anyone know of a way to avoid the With the current setup zigbee-herdsman-converters/src/devices/schneider_electric.ts Lines 2041 to 2049 in 014fb0c
The light entity in HA is named |
Got it, and it's working for the Swedish version as well. :) This is my device: This is the config
|
@FSund Could you please add support for the one I posted as well, or do you suggest a separate PR? |
@enoch85 I have not gotten on_level to work yet, but I suppose that's not required, and can be added later. Does all functionality work with your Exxact device? Ballast config, illuminance/lux detection etc? In that case I can add the Swedish device to this PR. |
Afaik, everything works. The only thing I need really is turning it off and on, and detect presence. Just tested ballast though, and it's also working! 👍🏼 Regarding the on_level I can see that it's on by a change in color on the switch. Please see pic for more info. |
Maybe would be nice with an icon of the device though, don't know if this PR covers that? |
I've pushed my PR which adds extends for ballast and the Schneider specific stuff. Everything should work, though I haven't 100% verified that yet. |
@peroo Your integration works perfect! I'm soo grateful, thank you! |
@@ -2128,6 +2128,36 @@ const definitions: DefinitionWithExtend[] = [ | |||
{vendor: 'Exxact', model: 'WDE003962'}, | |||
], | |||
}, | |||
{ | |||
zigbeeModel: ['NHMOTION/UNIDIM/1'], | |||
model: 'NHMOTION/UNIDIM/1', |
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 would expect a different model number is written on product page or device, can you use this?
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.
To be honest I'm not certain that this is actually a Schneider Electric white label product. It looks mostly identical to Elko EKO07250 and related devices, which is definitely produced by Schneider, but this device (EKO06984) seem like an earlier version.
In any case I haven't been able to find the Schneider product number anywhere.
I can change to model: 'EKO06984'
and vendor: 'Elko'
, but in that case I should perhaps move the definition over to elko.ts
. In that case I would also have to duplicate a lot of the schneiderElectricExtend
functionality.
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.
IMHO, why not add the device ID EKO06984
to the already existing models that were merged in #8300? Specifically here: https://github.com/Koenkk/zigbee-herdsman-converters/pull/8300/files#diff-9efd6085fa5f906d335c472b4ab9366dd0effdb29b6bbff96ef4033f4618daeeR2139-R2145
The functionality is probably exactly the same.
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 zigbeeModel
is different, NHMOTION/UNIDIM/1
vs. NHMOTION/DIMMER/1
.
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.
Then let's keep NHMOTION/UNIDIM/1
Could you also submit a picture for the docs? This can be done by:
Make sure that:
|
Docs and images was already added Koenkk/zigbee2mqtt.io#3285 But I see I perhaps used the wrong filenames. Should I rename the image and/or the markdown file? |
yes please rename |
I've combined the converters for Schneider WDE002961 (push button dimmer) and Schneider 545D6306 (PIR with relay) to come up with something that seems to work.
This supports on/off, dimming, ballast config, occupancy, and light level/illuminance.
Some testing still remains before it's ready for review.
Can't decide between ELKO or Elko, the database seems to have both entries.
TODO
0: RL-LED
and3: Auto
)0: RL-LED
and3: Auto
might be reversed)on_level
Optional(?)