-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Updates to match 2.14 firmware (previous 2.08) #5654
Conversation
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.
Apparently the 0-100 ranges @EricM_Inovelli provided for parameters 55 & 56 were a result of him lifting the parameter definitions directly from the Hubitat driver, which uses percentages rather than a 2-byte number converted to decimal. Since the z2m converter does not convert to percentages, the 0-100 range for parameters 55 and 56 is not correct. Please adjust the max value for parameters 55 and 56 to be 254. Forum user mamber tested all the way to 255, and found that the switch only responds if the values for these parameters is in the range 0-254. See this forum post: https://community.inovelli.com/t/blue-series-2-1-firmware-changelog-vzm31-sn/12326/425
Yup, I think 255 gets remapped to 254 in a few places?
zigbee-herdsman-converters/devices/inovelli.js Line 1061 in 1abb64d
|
@epower53 @InovelliUSA @edwardgarza Could you confirm if you actually see the new attributes in the 'Dev Console'? When I install the converter, it isn't able to pull these values. |
I don't see these in the Dev Console, either. I think that the manuSpecificInovelliVZM31SN cluster definition must also need an update. I had assumed/hoped that adding new parameters to the converter would do it, but it seems there's another file that needs additions. @InovelliUSA @edwardgarza can you confirm this is correct, and if so point us in the direction we need to go? |
@nathanfiscus confirms- we need to update the cluster definition. I don't have a good pull request capability... @michaelkkehoe would you mind submitting the PR for this? We need to add the new parameter definitions to zigbee-herdsman/src/zcl/definition/cluster.ts. The definition for manuSpecificInovelliVZM31SN starts on line 4926. |
@epower53 Yeah I'll start on it. |
Remove duplicate param 53
Datatypes look good. |
Thanks all! |
2.10
2.12
2.14