zwave-js / node-zwave-js

Z-Wave driver written entirely in JavaScript/TypeScript
https://zwave-js.github.io/node-zwave-js/
MIT License
751 stars 603 forks source link

Minimum value of Parameter range for HomeSeer Floodlight Sensor HS-FLS100+ is rejected #4187

Closed COCottonwood closed 2 years ago

COCottonwood commented 2 years ago

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?

NO, my problem is NOT within Home Assistant or the ZWave JS integration

Is your problem within ZWaveJS2MQTT?

NO, my problem is NOT within ZWaveJS2MQTT

Checklist

Describe the bug

Previous PR#3701 was CLOSED but NOT fixed. https://github.com/zwave-js/node-zwave-js/issues/3701

What causes the bug?

Unable to input parameter value of zero. It is still using MIN of 10 and MAX of 900

For HomeSeer HS-FLS100+, Lux_Trigger_Threshold, allowed values are 0, 30-200, 255. Value 0 will NOT activate floodlight. Value 30 represents 10 lux, Value 200 represents 900 lux. Value 255 will activate floodlight regardless of ambient light level

What do you observe?

Zwave JS HomeSeer HS-FLS100+ parameter options:

image

This does NOT match the Z-Wave JS Database

image

What did you expect to happen?

Per the HomeSeer HS-FLS100+ Manual (located at - https://products.z-wavealliance.org/ProductManual/File?folder=&filename=MarketCertificationFiles/2947/HS-FLS100-user-manual2b-A501112779R.pdf

image

What did you see?

image

Device information

Manufacturer: HomeSeer Model name: HS-FLS100+ Node ID in your network:

Currently on CORE 2022.2.1; SUPERVISOR 2022.01.1; HOME ASSISTANT OS 7.2

From Z-Wave log from RE-INTERVIEW:

2022-02-05T13:56:24.907Z CNTRLR [Node 046] Interview stage completed: NodeInfo 2022-02-05T13:56:24.912Z CNTRLR [Node 046] Interviewing Manufacturer Specific... 2022-02-05T13:56:24.913Z CNTRLR » [Node 046] querying manufacturer information... 2022-02-05T13:56:25.485Z CNTRLR [Node 046] [+] [Manufacturer Specific] manufacturerId: 12 [Endpoint 0] 2022-02-05T13:56:25.488Z CNTRLR [Node 046] [+] [Manufacturer Specific] productType: 513 [Endpoint 0] 2022-02-05T13:56:25.491Z CNTRLR [Node 046] [+] [Manufacturer Specific] productId: 11 [Endpoint 0] 2022-02-05T13:56:25.493Z SERIAL » [ACK] (0x06) 2022-02-05T13:56:25.498Z DRIVER « [Node 046] [REQ] [BridgeApplicationCommand] │ RSSI: -74 dBm └─[ManufacturerSpecificCCReport] manufacturer id: 0x0c product type: 0x0201 product id: 0x0b

How are you using node-zwave-js?

Which branches or versions?

version: node-zwave-js branch:

Diagnostics Driver Version: 8.10.2 Server Version: 1.14.1 Home ID: XXXXX Server URL: ws://core-zwave-js:3000

Did you change anything?

no

If yes, what did you change?

No response

Did this work before?

No, it never worked anywhere

If yes, where did it work?

No response

Attach Driver Logfile

No Log file needed

COCottonwood commented 2 years ago

FYI - I am Currently on CORE 2022.2.1; SUPERVISOR 2022.01.1; HOME ASSISTANT OS 7.2 today, and I re-interviewed the node, and still have this failure.

Do I need to remove this device and re-add it, because I would have to climb up on a ladder to re-add it as it is mounted on the corner of my house (and I have 5 of these)?

COCottonwood commented 2 years ago

PLEASE DO NOT TAKE ANY ACTION ON THIS YET... I might have opened this ISSUE too early.

I was just informed that I need to wait for the Z-Wave JS addon to pickup the right zwavejs version, then update it, then re-interview. See https://github.com/zwave-js/node-zwave-js/issues/3701

AlCalzone commented 2 years ago

Let me know how it goes

COCottonwood commented 2 years ago

Issue resolved, I did open this by mistake.