-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Multi-channel associations support #59
Comments
robertsLando
changed the title
Multi channel associations
Multi-channel associations support
Jun 27, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm tring to add/remove associations, my device has support for multi-channel associations but there is no way to specify the target instance using the APIs provided for group/associations management.
I explain my case: my device is a thermostat and has 5 instances:
In Open Zwave control panel I can set the target node instance using
<nodeID>.<instanceID>
notation but this seems not working using the openzwave-shared lib.I'm also tring to remove and add the controller to lifeline group of node 2 and I'm able to add it but not to remove it:
Removing controller (id 1) from lifeline group (1) of nodeid 2
Command used:
zwave.removeAssociation(2, 1, 1);
Add controller to lifeline group (1) of nodeid 2 (not working):
Command used:
zwave.addAssociation(2, 1, 1);
No error is thrown but the controller is not added to the group (as you can see from the getAssociations() call that result is [])
I think the main problem can be find in this lines:
Remove association:
2018-06-27 06:33:42.378 Info, Node002, MultiChannelAssociation::Remove - Removing instance 0 on node 1 from group 1 of node 2
Add association:
2018-06-27 06:45:05.246 Info, Node002, MultiChannelAssociation::Set - Adding instance 1 on node 1 to group 1 of node 2
The instance 1 of node 1 doesn't exist (as the controller is not multichannel instance) and there is no way to specify it using the APIs, the API should use by default instance 0 of controller and should also provide a way to specify it in the api call.
The apis should work like:
zwave.addAssociation(nodeid, group, target_nodeid.target_instanceID);
zwave.removeAssociation(nodeid, group, target_nodeid.target_instanceID);
if the
target_nodeid
hasn't the dot the default instance is used (root instance id 0).Here the link to multi instance support in open-zwave lib:
https://github.com/OpenZWave/open-zwave/blob/master/cpp/src/Group.cpp#L269
The text was updated successfully, but these errors were encountered: