andreypopov / node-red-contrib-zigbee2mqtt

Zigbee2mqtt connectivity nodes for node-red
GNU General Public License v3.0
94 stars 24 forks source link

Handle transition option differently #123

Open Cyberbeni opened 1 year ago

Cyberbeni commented 1 year ago

Currently the options are set permanently every time when you send message to an out node: https://github.com/andreypopov/node-red-contrib-zigbee2mqtt/blob/59d31200d8234ae4893aa66ba57e8c6913d26fdc/nodes/out.js#L217-L220

Transition option can be set for just a specific request: https://www.zigbee2mqtt.io/devices/LED2003G10.html#transition

I think that it would be better for most people to handle the transition option separately, so they can have their automatic brightness changes through Node-RED happen over some time while manual changes happen instantly.

chojnicki commented 8 months ago

I was confused why my light changed settings for timeout, and then checked source, logs and found this issue It's really weird idea to set options like transition for ever, when it is available for specific payload in Z2M like @Cyberbeni said. I guess that is the reason why transition cannot be set for devices group in this node.

I know this repo is not actively maintained anymore, but is there @andreypopov still a chance for improvements/fixes or you stopped using Z2M?

It is a great project, so it would be a shame to being forced to use standard mosquitto node for raw payloads.

Cyberbeni commented 8 months ago

It is a great project, so it would be a shame to being forced to use standard mosquitto node for raw payloads.

You can use raw payloads with this node, just set command to JSON and then payloads like this will only send the transition value for that 1 command: {"brightness":5,"transition":1}