home-assistant / home-assistant.io

:blue_book: Home Assistant User documentation
https://www.home-assistant.io
Other
4.74k stars 7.18k forks source link

Add missing IoT classes to integration documentation #14661

Closed frenck closed 3 years ago

frenck commented 3 years ago

Context

All our integration is classified with an IoT class.

Read more about this here:

https://www.home-assistant.io/blog/2016/02/12/classifying-the-internet-of-things/

Classifier Description
Assumed State We are unable to get the state of the device. Best we can do is to assume the state based on our last command.
Cloud Polling Integration of this device happens via the cloud and requires an active internet connection. Polling the state means that an update might be noticed later.
Cloud Push Integration of this device happens via the cloud and requires an active internet connection. Home Assistant will be notified as soon as a new state is available.
Local Polling Offers direct communication with device. Polling the state means that an update might be noticed later.
Local Push Offers direct communication with device. Home Assistant will be notified as soon as a new state is available.
Calculated The integration does not handle communication on it's own, but calculates the state.

For the future, we want to offer filtering and other indicators for this across the Home Assistant platform.

The problem

Not all our documented integrations have this IoT class set, and we need help completing that.

The below listed integration are all missing the ha_iot_class: in the top of their Markdown files, specifying the IoT class of this issue.

Valid values are:

ha_iot_class: Assumed State
ha_iot_class: Cloud Polling
ha_iot_class: Cloud Push
ha_iot_class: Local Polling
ha_iot_class: Local Push
ha_iot_class: Calculated

If you open up a PR for one of the integration listed below, please make sure to mention #14661 in your PR description so it gets linked to this issue. Thanks!

Integrations that are missing IoT class

The following integrations are missing an IoT class:

dasadi commented 3 years ago

@frenck What about internal integrations like "Alarm Control Panel" or "Alert"? From my understanding they are internal, which means they shouldn't have an associated IoT class. But in the list quite a few inegrations are internal and I am wondering what to do about them?

ZuluWhiskey commented 3 years ago

Out of interest, what is the idea with integrations like climate where the IoT class would depend on the device you're interfacing with? Or as it's an internal integration does it fall into the same category as (for example) Zone?

MatejMecka commented 3 years ago

I can't find Goal Zero Yeti, all the pages lead to a 404

frenck commented 3 years ago

@MatejMecka 0.116 😉 Will be available after today

MatejMecka commented 3 years ago

Ahh, ok ;) Can't wait

akubak commented 3 years ago

@frenck What about internal integrations like "Alarm Control Panel" or "Alert"? From my understanding they are internal, which means they shouldn't have an associated IoT class. But in the list quite a few inegrations are internal and I am wondering what to do about them?

@dasadi I asked the same thing in the Discord server and he said to use ha_iot_class: ~ for anything that doesn't have a class by itself, like a light or sensor. It's either that or ha_iot_class: Calculated if something gathers input from sensors and then calculates a state, which would apply to some of the alarm components.