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

Add deCONZ component documentation #3967

Merged
merged 36 commits into from
Jan 1, 2018
Merged

Add deCONZ component documentation #3967

merged 36 commits into from
Jan 1, 2018

Conversation

Kane610
Copy link
Member

@Kane610 Kane610 commented Nov 12, 2017

Description:
Component supporting deCONZ REST API exposing Zigbee devices such as Lights and Sensors.

Pull request in home-assistant: home-assistant/core#10321

Checklist:

  • New documentation for platforms/components and features should go to next.
  • The documentation follow the standards.

@Kane610 Kane610 changed the title WIP: Deconz component WIP: Deconz component documentation Nov 12, 2017
## {% linkable_title Configuration variables %}

- **host** (*Required*): The IP address to your deCONZ server.
- **api-key** (*Optional*): The username to your Axis device. Default is 'root'.
Copy link

Choose a reason for hiding this comment

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

Left overs from the Axis component ;)

## {% linkable_title Device services %}
Available services: `generate_api_key`.

#### {% linkable_title Service `deconz/generate_api_key` %}
Copy link

Choose a reason for hiding this comment

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

Please add a link to deconz documentation for this http://dresden-elektronik.github.io/deconz-rest-doc/getting_started/

@donnib
Copy link

donnib commented Nov 13, 2017

So here is some general feedback :

  • I think it would be VERY beneficial to make a list of the real devices supported since users will ask questions like does THIS specific device work ? so make a list similar to the one i started :
    https://bitbucket.org/mihaicph/deconz-custom-module/overview
  • Add examples of an event automation like i did in above link
  • Explain to users how they find the ID's in deconz by pointing to Postman for example (you can get some inspiration what i wrote)
  • Make a list of things that you know doesn't work at the moment or won't work when released such as renames in deconz only after reboot or what id deconz is rebooted, does it then still work, such things.
  • Also i think a best practice should be there such as explaining to users that they should try to make groups in deConz then control the groups when there is more than one light you wish to control at the same time, failing to do so can cause some weird Zigbee problems.

Added automation examples for a IKEA Trådfri dimmer and input number
Added list of supported device types
Added lists of verified to work devices
Removed references to Axis component

#### {% linkable_title Verified to be supported sensors %}

- IKEA Trådfri bulb GU10 W 400lm

Choose a reason for hiding this comment

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

Tested "TRADFRI bulb E27 WS�opal 980lm" as working. Not sure about the weird character in there but thats how it displays in the front end (and probably deconz to be fair)

Copy link
Member Author

Choose a reason for hiding this comment

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

Hmm, maybe I need to decode it to a specific type like UTF-8?


- **host** (*Required*): The IP address to your deCONZ server.
- **api-key** (*Optional*): The API key to your deCONZ server.
- **port** (*Optional*): Configure port deCONZ web server is accessible from. Default is 80.

Choose a reason for hiding this comment

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

I'm not sure if it's used here but in the more recent versions of deconz you can change the websocket port. Does this need to be taken into consideration here?

Copy link
Member Author

Choose a reason for hiding this comment

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

Websocket port is taken directly from Deconz, no need to configure.


- Open/close detection
- Presence detection
- IKEA Trådfri Motion Sensor
Copy link

Choose a reason for hiding this comment

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

Add HUE Motion sensor


[deCONZ](https://www.dresden-elektronik.de/funktechnik/products/software/pc/deconz/) by [Dresden Elektronik](https://www.dresden-elektronik.de) is a software that communicates with Conbee/Raspbee Zigbee gateways and exposes Zigbee devices that are connected to the gateway.

[deCONZ REST API](http://dresden-elektronik.github.io/deconz-rest-doc/) verified from v2.04.86.
Copy link

Choose a reason for hiding this comment

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

Change to 2.04.89

- Light level sensor
- Pressure sensor
- Switches
- IKEA Trådfri wireless dimmer
Copy link

Choose a reason for hiding this comment

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

@Kane610 Kane610 changed the title WIP: Deconz component documentation deCONZ component documentation Nov 21, 2017
@thoscut
Copy link
Contributor

thoscut commented Dec 13, 2017

Can confirm following devices to be working fine:

Xiaomi WXKG01LM Smart Home Wireless Switch
Xiaomi WXKG11LM Aqara Smart Wireless Switch
Xiaomi WXKG02LM Aqara Smart Light Switch
Xiaomi RTCGQ01LM Motion Sensor
Xiaomi RTCGQ01LM Smart Home Aqara Human Body Sensor
Xiaomi MCCGQ01LM Smart Home Security Door & Window Contact Sensor
Xiaomi WSDCGQ01LM MiJia Smart Temperature & Humidity Sensor
Xiaomi WSDCGQ11LM Aqara Temperature Humidity Sensor

Due to lack of support in deconz, battery values are still not reported.

@Kane610
Copy link
Member Author

Kane610 commented Dec 15, 2017

@tseel no battery support on any of those devices?

@thoscut
Copy link
Contributor

thoscut commented Dec 16, 2017

@Kane610 It's a known issue in the rest api of deconz and announced for the future, see dresden-elektronik/deconz-rest-plugin#13

@frenck frenck added the new-integration This PR adds documentation for a new Home Assistant integration label Dec 18, 2017
@frenck frenck self-requested a review December 19, 2017 22:44
Copy link
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

@Kane610 Requested some small change via review. Could you take a look?

---
layout: page
title: "deCONZ Binary Sensor"
description: "Instructions how to integrate Zigbee binary sensors from deCONZ into Home Assistant."
Copy link
Member

Choose a reason for hiding this comment

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

Instructions how to -> Instructions on how to


Home Assistant will automatically discover deCONZ presence on your network, if `discovery:` is present in your `configuration.yaml` file.

If you don't have the API key, you can generate an API key for deCONZ by using the one-click functionality similar to Philips Hue. Go to Menu->Settings->Unlock Gateway in deCONZ and then use the deCONZ configurator in HASS GUI to create an API key. When you've generated the API key from HASS the API key will be stored in deconz.conf inside the home-assistant folder.
Copy link
Member

Choose a reason for hiding this comment

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

  • Missing comma after HASS -> from HASS, the API
  • Replace HASS -> Home Assistant

| Parameter | Description |
|-----------|---------------------------------------------------------|
| `field` | String representing a specific device in deCONZ. |
| `data` | Data is a json object with what data you want to alter. |
Copy link
Member

Choose a reason for hiding this comment

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

json -> JSON


## {% linkable_title Configuration variables %}

- **host** (*Optional*): The IP address to your deCONZ server.
Copy link
Member

Choose a reason for hiding this comment

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

address to your -> address of your

- [Switches (Remote Controls)](/components/sensor/deconz/)
- [Temperature Sensors](/components/sensor/deconz/)

## {% linkable_title Configuration variables %}
Copy link
Member

Choose a reason for hiding this comment

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

Please use the configuration tags to display configuration variables. For more information see:
https://home-assistant.io/developers/documentation/create_page/#configuration

Copy link
Member

Choose a reason for hiding this comment

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

This header is not needed. It will be defined automatically when using the configuration tags.

footer: true
logo: deconz.jpeg
ha_category: Light
ha_release: "0.60"
Copy link
Member

Choose a reason for hiding this comment

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

bump to 0.61

footer: true
logo: deconz.jpeg
ha_category: Sensor
ha_release: "0.60"
Copy link
Member

Choose a reason for hiding this comment

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

bump to 0.61

---
layout: page
title: deCONZ Sensor
description: "Instructions how to integrate Zigbee sensors from deCONZ into Home Assistant."
Copy link
Member

Choose a reason for hiding this comment

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

Instructions how to -> Instructions on how to

---
layout: page
title: "deCONZ"
description: "Instructions how to integrate deCONZ scenes into Home Assistant."
Copy link
Member

Choose a reason for hiding this comment

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

Instructions how to -> Instructions on how to

footer: true
logo: deconz.jpeg
ha_category: Scene
ha_release: "0.60"
Copy link
Member

Choose a reason for hiding this comment

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

bump to 0.61

Copy link
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Click approve instead of request changes (correcting)

@Kane610
Copy link
Member Author

Kane610 commented Dec 20, 2017

@frenck thanks for feedback! I followed your suggested improvements.

frenck
frenck previously approved these changes Dec 20, 2017
Copy link
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

Thanks @Kane610 🏅

@Kane610
Copy link
Member Author

Kane610 commented Dec 20, 2017

Much appreciated @frenck !

- [Switches (Remote Controls)](/components/sensor/deconz/)
- [Temperature Sensors](/components/sensor/deconz/)

## {% linkable_title Configuration variables %}
Copy link
Member

Choose a reason for hiding this comment

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

This header is not needed. It will be defined automatically when using the configuration tags.


| Parameter | Description |
|-----------|---------------------------------------------------------|
| `field` | String representing a specific device in deCONZ. |
Copy link
Member

Choose a reason for hiding this comment

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

Please format as our standard service table. Also extend this table with info on if parameter is optional. See eg:
https://home-assistant.io/components/media_player.sonos/#service-sonos_snapshot

#### {% linkable_title Service `deconz/configure` %}
Set attribute of device in Deconz using [Rest API](http://dresden-elektronik.github.io/deconz-rest-doc/rest/).

| Parameter | Optional | Description |
Copy link
Member

Choose a reason for hiding this comment

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

Please rename Parameter to Service data attribute.

Copy link
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Thanks!

@MartinHjelmare MartinHjelmare changed the title deCONZ component documentation Add deCONZ component documentation Jan 1, 2018
@Kane610
Copy link
Member Author

Kane610 commented Jan 1, 2018

Thanks a lot @MartinHjelmare !

@MartinHjelmare MartinHjelmare merged commit 943b2e3 into home-assistant:next Jan 1, 2018
@Kane610 Kane610 deleted the deconz-component branch February 6, 2019 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-integration This PR adds documentation for a new Home Assistant integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants