Skip to content
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

Rewriting the Z-Wave documentation #3467

Merged
merged 6 commits into from
Oct 14, 2017
Merged

Conversation

DubhAd
Copy link
Contributor

@DubhAd DubhAd commented Sep 30, 2017

This is a starting point for rewriting the Z-Wave documentation, to reflect all the undocumented changes, addition of the control panel, etc.

Discussion and feedback appreciated ;)

Copy link
Contributor

@turbokongen turbokongen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good rewrite!
Only small changes needed.


### {% linkable_title Node config options %}

You can set the *wakeup* interval (in seconds) of the device, this is shown for all devices, but only applies to battery powered devices.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is only shown for devices reporting back the wakeup command class. Thus should only be battery powered devices.


## {% linkable_title Polling %}

Where a device doesn't send updates on status changes to the controller, you can work around this by using a thing called Polling. That causes the controller to ask the device to provide an update on all its sensors and states. This will cause a lot of traffic on the network, and if you poll too many devices too quickly, you can effectively break your Z-Wave network. Polling should only be used where there is no other choice, and you should use as large a polling interval as possible. Ideally you should replace the device.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add an example here to how this works:
If polling_interval=60000 (which is default) and you have 10 values that are being polled, if a device can receive and aknowledge the poll within a second, it takes 10 seconds to complete the polling list, which leaves 50 seconds left for normal traffic. So the more polling are done, the less bandwidth is available.

Or something similar to that :)

---

<p class='note'>
This is a work in progress, based upon reports in the forum, the author's own devices, and reading of various documentation. It will be incomplete, so if you have a device that isn't reported here, or have a device that reports a different value, please provide a report in the [Z-Wave section](https://community.home-assistant.io/c/configuration/zwave) of the forum or the #zwave channel on [Discord](https://discord.gg/RkajcgS).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I'll use that for the next round of updates

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be great if this doc was linked from the main zwave page. I just found it after using zwave since day one and its a treasure trove of in information! Great job!

basic_level: 255
```

The *object_id* and *basic_level* of all triggered events can be seen in the console output.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

entity_id not object_id

scene_data: 3
```

The *object_id* and *scene_id* of all triggered events can be seen in the console output.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

entity_id

DubhAd added 3 commits October 6, 2017 07:12
Note that it only applies to devices that can be battery powered (for example, my Aeotec sensor is USB powered, but as it supports Wake the option is shown)
Example of what the impact of polling can be
@turbokongen
Copy link
Contributor

Perfect! Just some conflicts that needs to be resolved before we can merge.

@DubhAd
Copy link
Contributor Author

DubhAd commented Oct 14, 2017

Anything I can do to help this further along?

@turbokongen
Copy link
Contributor

Yes, there are conflicting changes that needs to be resolved before we are able to merge.

@DubhAd
Copy link
Contributor Author

DubhAd commented Oct 14, 2017

Thanks, first time with conflicts on Github so didn't know if it was something I could fix, or not.

@turbokongen
Copy link
Contributor

Thanks💃

@turbokongen turbokongen merged commit 018890a into home-assistant:next Oct 14, 2017
@Danielhiversen Danielhiversen mentioned this pull request Oct 16, 2017
---

<p class='note'>
This is a work in progress, based upon reports in the forum, the author's own devices, and reading of various documentation. It will be incomplete, so if you have a device that isn't reported here, or have a device that reports a different value, please provide a report in the [Z-Wave section](https://community.home-assistant.io/c/configuration/zwave) of the forum or the #zwave channel on [Discord](https://discord.gg/RkajcgS).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be great if this doc was linked from the main zwave page. I just found it after using zwave since day one and its a treasure trove of in information! Great job!

sensor_class: opening
friendly_name: "Friendly name here"
value_template: >-
{%- if is_state('sensor.YOUR_SENSOR_access_control', '22') -%}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think access_control should have the if state written to 23 - closed. This was if it's not 23, it's not closed.

Otherwise, either 254 or 255 would show as closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants