-
-
Notifications
You must be signed in to change notification settings - Fork 603
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
ZMNHDA Qubino Flush Dimmer multilevel switch current value not propagating from zwavejs to zwavejs2mqtt #2910
Comments
@AlCalzone ? I don't think I'm missing the value event |
@NINE78 I see also the endpoint 0 updates in your logs:
|
Value added for 64-38-0-currentValue, but it's never updated, whereas zwavejs is picking up changes on both endpoints... Do note timestamps on my paste above; don't think this should be closed. Thanks! |
I think that is a misunderstanding. The device reports to endpoint 0 (which duplicates endpoint 1 and thus is hidden). Because you still want to see the value, the report is mapped to endpoint 1 - and that report is passed on to zjs2mqtt. |
Ah ok, thanks @AlCalzone. So it appears Endpoint 1 isn’t propagated to Home Assistant. I notice only Endpoint 0 is added on interview, and Endpoint 1 Currrent Value when it is first transmitted (as observed in zwavejs2mqtt). Is this expected behavior? Trying to figure out why the device in HA isn’t reflecting state. Is there a way to force Endpoint 1 in the config upon interview? |
@robertsLando from the log in the first comment it looks like the value reaches zjs2m, so the problem probably lies between there and HA, right? |
@NINE78 Are you using mqtt discovery or zwavejs server ? |
@robertsLando Zwavejs server to which HA integration connects over websocket. So my current thinking is that:
Does that make sense? From his comment above, @AlCalzone suggests that — due to duplication — endpoint 0 is hidden. Can we somehow force this not to? Or alternatively force xx-38-1-currentValue to be added during the interview process? |
The latter is what I'd expect to happen. Can you re-interview the device and share a log of that? Can't debug this when the interview restarts from cache. |
Just performed a re-interview of the device, followed by a toggle off and on again (the light was on when re-interviewing). From the log I observe, when toggling the light for the first time: Endpoint 0 is updated, and Endpoint 1 currentValue appears to be newly created:
Thanks so much! |
I have the same problem with all my Qubino Flush Dimmers. When the wall switch is toggled, the "dimmer level" turns on, but the switch in HA stays off. When I turn the "dimmer level" or the switch on in lovelace, the light turns on as it should. However - the big problem is that when turning off the light with the "dimmer level" button in lovelace, the light turn off but after a second or so, the "dimmer level" button comes on again (with the light still off) creating a lot of issues with wrong status of lights. If the light is turned off with the switch in HA, it turns off as it should and the status of the light is off in lovelace. |
Same for me on ZMNHDD dimmers. Is there anything that I can do to help in improving the situation? |
@AlCalzone Updates on this? I think it's something on your side |
Essentially this is a mix of #2286 and automatically figuring out the correct associations. There's no one-fits-all solution, so I've been putting this off while thinking about an approach that requires as few device-specific overrides as possible. |
This should be fixed with #2997 in the upcoming 8.x release. If not, feel free to reopen with new logs. |
Also seeing this issue with all ZMNHDD. Have re-interviewed the devices but still the same issue when turning off the lights. Turning on works fine. Currently using z-wave js addon in HA 0.1.32 which includes Z-Wave JS 8.0.5 |
Z-Wave JS 8.x includes some changes to endpoints/associations which might require compat flags for some devices. Please share a driver log of a re-interview, so I can use that to figure out which changes are necessary. |
Is "driver log" the same as I get from the Home Assistant addon for z-wave js? Is there a way to filter the logs for a specific node? I have a lot of nodes reporting all the time which clutter the logs. With which serviery do you what the log? Debug? Info? |
Its this one - loglevel "debug" please:
zwavejs2mqtt should let you do that. If not, it's also okay. |
Hope this helps. |
Not sure if this also helps, but it seems like if i start with what I think is brightness level 50 and turn off the light, its correctly reflected in the UI. However, If i start with brightness level 30, its not working. it will still be turned on in the UI. |
At first I missed the exclusion and reinclusion part. Then I was even not able to see the Binary Swicth |
OK I did a re-interview and here are the log and node.json As far as I understand in the log the endpoint for the switch was found .. |
Ok we can work around that with a compat flag. Will add it later. |
@AlCalzone so I maybe find something that can be fixed ? |
Hello again, I have Save -restart (even reboot) - reinterviewed. zwavejs2mqtt: 5.8.0 @robertsLando did I miss something to activate custom config file ? |
Seems you have followed all the steps described in: https://zwave-js.github.io/zwavejs2mqtt/#/device-support/custom-device-files Just found the error in logs: 2021-10-18T11:04:38.140Z CONFIG Error parsing config file zmnhtd.json: The device config file is malformed!
packages/config/config/devices/zmnhtd.json:
paramInformation is not an object |
indeed I didn't notice this message :-(
Reading the documentation : https://zwave-js.github.io/zwavejs2mqtt/#/device-support/custom-device-files
From my undersatnding : paramInformation is not an object but an array (so I'm not sure the error message is accurate) |
The problem is that we recently switched the config file format. You copied a new file which is not yet understood by the driver version installed on your system. |
@AlCalzone If beta 2 supports it he can use |
yeah beta supports it |
@fbietlot So just use |
Regarding Qubino's response
They are confusing decimal 37 (hexadecimal |
What build method are yopu using? If docker instead of using |
I'm just clicking on install in Home Assistant Supervised 😳 😕 |
Oh ok so you are using hass addon, @blhoward2 how can he use docker images with hass os? |
The only way to use a built docker image is by installing the Portainer addon, or you may be able to install the ssh addon and manually run docker (I'm not sure on that one). Otherwise he's limited to the zwavejs addon or the zwavejs2mqtt addon. https://community.home-assistant.io/t/home-assistant-community-add-on-portainer/68836 |
Hi to all, yes I did assume that the command you're sending was in RAW so in hexadecimal, that's why the mistake. I didn't use the zvaveJS to zwave2mqtt as I was testing and checking the command with the PC controller and the Zniffer to see what is going on between the device and the z-wave controller. As you may see from the video and the Gif the Smart Meter reacts and replies properly if the proper command is received. @NINE78 I went again to the first message you send us and notice that you use the Parameter 100 set to 1 while the IKA is controlled setting the parameter 100 to 2 After you do the change exclude the Smart Meter, wait at least 30 seconds and re-include it again. |
Hi @QubinoHelp thank you for the follow up. I understand the confusion. "label": "EP IR external relay dsbld, External relay enbld", I will try to install PC controller to also do some testing. But unfortunately I don't think I have the knowledge to deal with a Zniffer.
I see some difference between your command (but not yet able to decode this) |
It looks like your commanding false and it's reporting that it's already false. What happens if you set true? |
sorry not the best example (I was focusing on message structure.)
|
@fbietlot please select your logs after pasting and click this button: |
Me again but with good news this time ;-) Here is the config
Versus the manual
Value 1 and 2 are inverted. |
That file needs to be reworked as a whole, but I'll at least fix this switchup for now. Great that you figured it out. |
Version
Build/Run method
zwavejs2mqtt version: 1.1.0
zwavejs version: 6.1.3
Describe the bug
For my Qubino (GOAP) ZMNHDA dimmers, I see multilevel switch ("current value") updates sent to Endpoints 0 and 1, but only Endpoint 1 is received in zwavejs2mqtt, this when manually toggling the light switch.
From zwavejs log:
However in zwavejs2mqtt log, only 38-1 value is updated, which does not get picked up by the "Current Value" entity in HA (there is only one for the light):
2021-02-10 15:34:46.411 INFO ZWAVE: Node 64: value updated: 38-1-currentValue 63 => 0
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Also expect 64-38-0-currentValue to be updated
Additional context
Logs attached
zwavejs2mqtt.log
zwavejs_578.log
The text was updated successfully, but these errors were encountered: