dvd-dev / hilo

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

Fix potentiel pour energy sensors #308

Closed ic-dev21 closed 11 months ago

ic-dev21 commented 11 months ago

Possible fix pour #121 #281 #283 #292

Réordonné la création des entités pour créer les power sensors plus tôt dans le boot pour aider les riemann sum.

Mis la bonne unité dans les energy sensors de base. Je ne voyais pas l'utilité de garder des Wh.

Retrait de async_get_last_state pour les energy sensors, causait des problèmes souvent au redémarrage. Amenait des state unavailable aussitôt qu'il y avait un petit rot chez Hilo ou un manque de données temporaire. Les sensors devenaient alors unavailable sans moyen de les ramener. Les riemann sum tombait donc indisponible et inutilisables.

@valleedelisle je pense avoir causé du dead code par la bande ligne 278. Code review si jamais tu as le temps?

Il pourrait être nécessaire de faire le bouton fix issue dans developper tools -> statistics pour régler les unités.

ic-dev21 commented 11 months ago

J’espère que je l’ai réglé pour un bout cette portion du code là

Le sam. 28 oct. 2023 à 06:53, David Vallee Delisle @.***> a écrit :

Merged #308 https://github.com/dvd-dev/hilo/pull/308 into main.

— Reply to this email directly, view it on GitHub https://github.com/dvd-dev/hilo/pull/308#event-10797605544, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZZGCFI3VE77I2JGMLLUSLTYBTPZDAVCNFSM6AAAAAA6TQD472VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQG44TONRQGU2TINA . You are receiving this because you authored the thread.Message ID: @.***>