Incorrect values set when sending negative values via controller

[bakman2]

Sending -20, device gets updated correctly.
Sending -12, devices get updated with -11

Debug Log

2021-10-17T20:06:59.588Z DRIVER « [Node 002] [REQ] [ApplicationCommand]
                                  └─[ConfigurationCCReport]
                                      parameter #: 8
                                      value size:  1
                                      value:       -20
2021-10-17T20:07:02.229Z SERIAL « 0x010c00040002063105014208a925                                      (14 bytes)
2021-10-17T20:07:02.233Z CNTRLR   [Node 002] [Multilevel Sensor] Air temperature: metadata updated  [Endpoint 0]
2021-10-17T20:07:02.233Z CNTRLR   [Node 002] [~] [Multilevel Sensor] Air temperature: 23.06 => 22.1 [Endpoint 0]
                                  7
2021-10-17T20:07:02.234Z NDERED » [Node: 2] [VALUE_UPDATED] [DIRECT] Forwarding payload...
2021-10-17T20:07:02.234Z NDERED » [Node: 2] [VALUE_UPDATED] [EVENT] Forwarding payload...
2021-10-17T20:07:02.235Z SERIAL » [ACK]                                                                   (0x06)
2021-10-17T20:07:02.236Z DRIVER « [Node 002] [REQ] [ApplicationCommand]
                                  └─[MultilevelSensorCCReport]
                                      type:  Air temperature
                                      scale: Celsius
                                      value: 22.17
2021-10-17T20:07:08.129Z NDERED « [CONTROLLER] Payload received.
2021-10-17T20:07:08.129Z NDERED « [Node: 2] [API: ValueAPI] [Method: setValue]
                                  └─[params]
                                      0: {"commandClassName":"Configuration","commandClass":112,"endpoint":0,"pr
                                  operty":8,"propertyName":"Temperature Offset"}
                                      1: -12
2021-10-17T20:07:08.141Z SERIAL » 0x010c0013020570040801f4250942                                      (14 bytes)
2021-10-17T20:07:08.142Z DRIVER » [Node 002] [REQ] [SendData]
                                  │ transmit options: 0x25
                                  │ callback id:      9
                                  └─[ConfigurationCCSet]
                                      parameter #:      8
                                      reset to default: false
                                      value size:       1
                                      value format:     SignedInteger
                                      value:            -12
2021-10-17T20:07:08.143Z SERIAL « [ACK]                                                                   (0x06)
2021-10-17T20:07:08.150Z SERIAL « 0x0104011301e8                                                       (6 bytes)
2021-10-17T20:07:08.151Z SERIAL » [ACK]                                                                   (0x06)
2021-10-17T20:07:08.151Z DRIVER « [RES] [SendData]
                                    was sent: true
2021-10-17T20:07:09.433Z SERIAL « 0x010700130900008163                                                 (9 bytes)
2021-10-17T20:07:09.434Z SERIAL » [ACK]                                                                   (0x06)
2021-10-17T20:07:09.435Z DRIVER « [REQ] [SendData]
                                    callback id:     9
                                    transmit status: OK
2021-10-17T20:07:10.443Z SERIAL » 0x010a00130203700508250ab5                                          (12 bytes)
2021-10-17T20:07:10.444Z DRIVER » [Node 002] [REQ] [SendData]
                                  │ transmit options: 0x25
                                  │ callback id:      10
                                  └─[ConfigurationCCGet]
                                      parameter #: 8
2021-10-17T20:07:10.446Z SERIAL « [ACK]                                                                   (0x06)
2021-10-17T20:07:10.452Z SERIAL « 0x0104011301e8                                                       (6 bytes)
2021-10-17T20:07:10.452Z SERIAL » [ACK]                                                                   (0x06)
2021-10-17T20:07:10.453Z DRIVER « [RES] [SendData]
                                    was sent: true
2021-10-17T20:07:10.473Z SERIAL « 0x010700130a000002e3                                                 (9 bytes)
2021-10-17T20:07:10.474Z SERIAL » [ACK]                                                                   (0x06)
2021-10-17T20:07:10.474Z DRIVER « [REQ] [SendData]
                                    callback id:     10
                                    transmit status: OK
2021-10-17T20:07:10.507Z SERIAL « 0x010b000400020570060801f57d                                        (13 bytes)
2021-10-17T20:07:10.509Z CNTRLR   [Node 002] [~] [Configuration] 8: -12 => -11
2021-10-17T20:07:10.510Z NDERED » [Node: 2] [VALUE_UPDATED] [DIRECT] Forwarding payload...
2021-10-17T20:07:10.510Z NDERED » [Node: 2] [VALUE_UPDATED] [EVENT] Forwarding payload...
2021-10-17T20:07:10.511Z SERIAL » [ACK]                                                                   (0x06)
2021-10-17T20:07:10.513Z DRIVER « [Node 002] [REQ] [ApplicationCommand]
                                  └─[ConfigurationCCReport]
                                      parameter #: 8
                                      value size:  1
                                      value:       -11

[marcus-j-davies]

@AlCalzone - anything of interest?

2021-10-17T20:07:08.142Z DRIVER » [Node 002] [REQ] [SendData]
                                  │ transmit options: 0x25
                                  │ callback id:      9
                                  └─[ConfigurationCCSet]
                                      parameter #:      8
                                      reset to default: false
                                      value size:       1
                                      value format:     SignedInteger
                                      value:            -12

2021-10-17T20:07:10.513Z DRIVER « [Node 002] [REQ] [ApplicationCommand]
                                  └─[ConfigurationCCReport]
                                      parameter #: 8
                                      value size:  1
                                      value:       -11

https://devices.zwave-js.io/?jumpTo=0x0148:0x0003:0x0001:0.0

I am not sure what their manual is showing

[AlCalzone]

0xCE is the hexadecimal 1-byte representation of the two's complement of -50, 0x32 is +50

Here's the data that gets sent for -12

SERIAL » 0x010c0013020570040801f4250942
                               ^^ - This is -12!

This comes back:

SERIAL « 0x010b000400020570060801f57d
                                 ^^ - This is -11!

I don't see any bug on our side (and it would be surprising to find one at this time). I'd say the device has issues with the negative representation but then the -20 is handled correctly... 🤷🏻‍♂️

Honestly, this is a question for the manufacturer support.

[marcus-j-davies]

Thanks,

I have configs which use negative values - with no issue, but wanted your opinion in case I missed something.

@bakman2,
One for the manufacture - they may have a firmware update potentially, that may have addressed this behaviour (if indeed they are aware)