homeassistant-zwave_mqtt icon indicating copy to clipboard operation
homeassistant-zwave_mqtt copied to clipboard

Add services

Open cgarwood opened this issue 4 years ago • 6 comments

Add services for various OpenZWave functions.

  • [x] add_node #35
  • [x] add_node_secure #35
  • [x] remove_node #35
  • [x] cancel_command #35
  • [ ] heal_network
  • [x] set_config_parameter
  • [x] remove_failed_node #35
  • [x] replace_failed_node #35
  • [ ] refresh_node_value
  • [ ] refresh_node
  • [ ] set_wakeup
  • [ ] reset_node_meters
  • [ ] heal_node
  • [ ] test_node
  • [ ] set_poll_intensity others?

Unlike the current zwave component implementation of these services, we'll need to also allow specifying an OZW Instance number since multiple OZW instances can be running and reporting to MQTT. We should default the OZW Instance to be instance 1 though, for compatibility.

May require upstream modifications to the python-openzwave-mqtt library

cgarwood avatar Dec 28 '19 02:12 cgarwood

completed:

add_node (with secure param) remove_node cancel_command set_config_parameter remove_failed_node replace_failed_node

to do:

refresh_node_value refresh_node refresh_entity heal_network set_wakeup reset_node_meters heal_node test_node set_poll_intensity change_association set_node_value print_config_parameter print_node

marcelveldt avatar Jan 21 '20 11:01 marcelveldt

set_config_parameter should probably be reviewed to make it a bit more user friendly. now the only possibility is to set a value by index and value_id.

We can also add support for setting a value by name and label, which is a bit more user friendly.

marcelveldt avatar Feb 25 '20 13:02 marcelveldt

I think we can leave the following services in the to-do list out, as they can be handled through the OZW admin tools if needed, and should be websocket commands when we build the new management UI in HA: heal_network set_wakeup heal_node test_node set_poll_intensity change_association set_node_value print_config_parameter print_node

That would leave the following on the todo list: refresh_node_value refresh_node refresh_entity reset_node_meters

cgarwood avatar Apr 30 '20 16:04 cgarwood

FYI, none of the network commands are implemented in OZW Admin yet.

kpine avatar Apr 30 '20 18:04 kpine

set_config_parameter was added here, but not in the core component. I would find that particularly useful for automating a few devices, such as changing the default brightness of lights. I'm wondering what needs to be done to bring that particular service over? Should that be discussed here or as a core issue? I'd be happy to open an issue and get that discussion started there.

uSpike avatar Jun 16 '20 02:06 uSpike

I think many people use that feature. It will be added, on our todo list.

marcelveldt avatar Jun 16 '20 05:06 marcelveldt