dvd-dev / hilo

Home Assistant Hilo Integration via HACS
MIT License
122 stars 26 forks source link

Les entities pour les thermostats Hilo ne contiennent pas de propriété énergie (energy) #353

Closed DoumLaberge closed 5 months ago

DoumLaberge commented 5 months ago

Je regardais la documentation zigbee pour les thermostats HILO et ils semblent avoir la possibilité de voir "energy" en plus du power.

Dans la fonction generate_entities_from_device, il n'y a pas de if device.has_attribute("energy"): Est-ce que c'est Hilo qui ne donnent pas accès a l'information, ou simplement un oubli?

Documentation:

Power (numeric) Instantaneous measured power. Value can be found in the published state on the power property. It's not possible to read (/get) or write (/set) this value. The unit of this value is W.

Energy (numeric) Sum of consumed energy. Value can be found in the published state on the energy property. It's not possible to read (/get) or write (/set) this value. The unit of this value is kWh.

ic-dev21 commented 5 months ago

Actuellement c’est géré via la case “generate energy meters” de la config. Je m’avance mais que je sache ça n’est pas exposé par Hilo

DoumLaberge commented 5 months ago

Le generate energy meters, ca ne fait que simplifie l'affichage de meter ( de ce que je comprends) mais en dessous ca demeurent la propriété "power" et non "energy" Edit: On dirait que le type Meter accumule en kwh, ce qui serait l,équivalent du "energy'

ic-dev21 commented 5 months ago

Le generate energy meters, ca ne fait que simplifie l'affichage de meter ( de ce que je comprends) mais en dessous ca demeurent la propriété "power" et non "energy"

Non, ça génère les entités nécessaire pour pouvoir les mettre dans le dashboard energy. C'est à ça que sert l'entité après tout.

Pour voir le "energy" dans le device, faut que le device "expose" cette propriété là à HA. Actuellement, Hilo ne fait pas de mesure en kWh par device, seulement sur le meter. C'est pourquoi ça n'est pas dans le device.

Je te dis pas que c'est pas faisable, juste que ça impliquerait de l'inclure dans le device lors de sa création par l'intégration. J'pas assez ferré pour te dire si c'est simple ou non à faire ou même faisable par contre.

DoumLaberge commented 5 months ago

De ce que je comprends du code, c'est que si la propriété est exposé par l'API Hilo ( car on ne parle pas directement au thermostat), c'est super simple de l'ajouter dans le device.

ic-dev21 commented 5 months ago

De ce que je comprends du code, c'est que si la propriété est exposé par l'API Hilo ( car on ne parle pas directement au thermostat), c'est super simple de l'ajouter dans le device.

C'est ce que je te dis, elle ne l'est pas. C'est pour ça que la consommation par device n'est pas dispo dans l'app native Hilo non plus.

FrancoLoco commented 5 months ago

C'est moi qui a codé la version Zigbee2MQTT. Quand je me suis connecté direct au thermostat j'ai vu que celui-ci poussait cette info... mais Hilo, ne semble pas partager cette donnée quand tu passes par leur système (ils la recueillent probablement mais la gardent pour eux selon moi).

DoumLaberge commented 5 months ago

@ic-dev21 C'est la phrase " Je m’avance mais que je sache ça n’est pas exposé par Hilo" qui m'a mêlé dans votre intervention. ;o)

ic-dev21 commented 5 months ago

@ic-dev21 C'est la phrase " Je m’avance mais que je sache ça n’est pas exposé par Hilo" qui m'a mêlé dans votre intervention. ;o)

J'avoue haha, mais je voulais dire pas exposé par Hilo Hilo, pas par l'intégration

ic-dev21 commented 5 months ago

Closed as not planned