ryanjohnsontv / node-red-contrib-ha-inovelli-manager

Nodes for easily managing scenes, parameters, and notifications of Inovelli switches paired through various Home Assistant integrations
MIT License
15 stars 3 forks source link

Dynamic node_id? #16

Closed SlothCroissant closed 3 years ago

SlothCroissant commented 3 years ago

Hey there, love the work you've done here to radically simplify scene management with Inovelli switches! Quick ask - is it possible to use dynamic values for the node_id in the scene manager node? Ref:

image

When I check the value of msg.payload.event.node_id using a debug node, it shows the proper node_id integer.

image

And when I manually enter that integer into the scene manager config, it works great. But when I try to use msg.payload.event.node_id in the node config, I can't seem to get it to work.

Any advice? Thanks!

ryanjohnsontv commented 3 years ago

Currently that field uses the provided node ID, or a list of node IDs, to send the value notification message to the proper node output. Perhaps I'm not understanding, but in its current state leaving that wide open it would try to process every single Z-Wave value notification, meaning if you have different switch types in your Z-Wave network then they would be incorrectly sorted through the node. I'm very interested to understand your use case because I want this to be one-size-fits-all solution for people. If it's of any interest, for my set up I have all of my dimmer node IDs in a dimmer scene-manager node and each output of that node feeds to a specific function node which then handles automatons based on the msg.payload.event.node_id value. Not sure if that's of any help to you Screen Shot 2021-09-26 at 10 07 04 PM .

SlothCroissant commented 3 years ago

Thanks for the reply! My use case may be somewhat unique, in that the only scene capable devices in my network are LZW30-SN switches.

So I'd love to be able to have options for "if ANY device is 5x tapped, do X" situations without having to hard code node IDs wherever possible. Or some other upstream filtering that's not dependent on hard coded node ID.

Hope that helps clarify!

ryanjohnsontv commented 3 years ago

Brilliant, gotcha! I'm wrapping up for the evening, but tomorrow I can add a passthrough toggle that disables the Node ID field and just sends through any value notification message.

SlothCroissant commented 3 years ago

Awesome! Thanks for the quick turnaround! Once you're set, let me know and I'll be happy to test and confirm for ya.

ryanjohnsontv commented 3 years ago

Just released! Let me know if you have any issues. https://github.com/ryanjohnsontv/node-red-contrib-ha-inovelli-manager/releases/tag/v0.1.1

SlothCroissant commented 3 years ago

Works like a charm!

I can now use a switch statement to set a variable of target devices, and the one node processes them all:

image