Incompatibilité core-2022.2.0b0 et hilo_energy_total_daily_low?

[FrancoLoco]

J’ai updaté à core-2022.2.0b0 et j’ai des problèmes avec mes compteurs de énergie, ça fait des erreurs avec le hilo_energy_total_daily_low

c’est comme si ça a été renommé ou ça n’existe plus.. entk dans le code chez nous j’ai mis hilo_energy_total_low et ca semble régler à court terme..

mais je vous avise si jamais vous pensiez updater à la nouvelle bêta core ça pourrait briser et vous forcer à taponner… à suivre

[FrancoLoco]

Ouin finalement j’ai restoré mon backup de la version stable du core, car les trucs de énergie semblent avoir pas mal changé… je ne sais pas si la version finale restera comme ça, mais présentement ça ne marche pas tel quel avec la version actuelle du code de l’intégration Hilo…

[valleedelisle]

J'vais tenter un update en fin de semaine et voir comment ça va. C'est possible que le code des utility_meters a changé chez Hass, c'est une composante qui est encore à sa phase enfant.

[FrancoLoco]

Je confirme que le problème est encore là avec la version finale de home assistant core 2022.2 qui vient d’être releasée.. rollbacking again..

[valleedelisle]

Je viens d'essayer 2022.2:

2022-02-02 19:38:48 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up hilo platform for sensor
Traceback (most recent call last):
  File "/home/dvd/hass/core/.venv/lib/python3.9/site-packages/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
    await asyncio.shield(task)
  File "/home/dvd/hass/core/config/custom_components/hilo/sensor.py", line 186, in async_setup_entry
    await utility_manager.update(async_add_entities)
  File "/home/dvd/hass/core/config/custom_components/hilo/managers.py", line 66, in update
    await utility_setup_platform(
  File "/home/dvd/hass/core/.venv/lib/python3.9/site-packages/homeassistant/components/utility_meter/sensor.py", line 107, in async_setup_platform
    for conf in discovery_info.values():
AttributeError: 'list' object has no attribute 'values'

[valleedelisle]

C'est le commit en question qui a tout brisé: home-assistant/core@58a5304

[FrancoLoco]

Good find!

[skimaniac]

je crois que ça cause le même problème avec sensor.hilo_energy_total_daily_medium . Avant la dernière mise a jour de Hilo il y avait le low et médium en erreur.

[FrancoLoco]

Ouin j’ai aussi le problème je reopen la issue

[FrancoLoco]

En fait il faudrait que @valleedelisle le fasse on dirait

[valleedelisle]

ouais je crois que j comprends le problème.

[valleedelisle]

Non c'est pas bon

[valleedelisle]

Ça marche mais le nom change. J'essaie de voir de ou vient ce changement...

[valleedelisle]

J'pense ben je l'ai eu, j'devrais tester un peu plus avant de push, ou ca prendrait un peu de CI/CD.

[skimaniac]

Je veux bien t'aider, mais j'ai pas de CI/CD j'ai juste de la Bud/Bud Light 😂

[valleedelisle]

Je veux bien t'aider, mais j'ai pas de CI/CD j'ai juste de la Bud/Bud Light joy

toi tu te magazines des bugs 🤣

[skimaniac]

Ok pas de Bud Light, mais je devrait t'en payer une... ça marche 👍🏻

[FrancoLoco]

Pas besoin de CI/CD tu as une petite armée de testeurs qui guette comme des fauves! On est assoiffés d’updates!

[arsenicks]

Est-ce que ca mérite de faire un pre-release ou on attend d'avoir mergé #85 ?

[FrancoLoco]

Moi ça ne marche toujours pas

[arsenicks]

crap, j'ai updaté à 2022.2.0 en pensant que ca fixait et qu'il était dans le dernier release

[skimaniac]

moi comme je sais pas trop quoi faire avec des .diif etc, j'ai juste fait du copié/collé du code de managers.py (c'est probablement pas trop la meilleur façon de faire) Mais tout fonctionne à merveille depuis hier soir, j'ai sensiblement les même données de consommation en HA et l'app Hilo.
Je vais retourné à mes lectures et video d'explication de CI/CD 🤓

[arsenicks]

C'est quoi la facon la plus simple de tester des fix comme ca pour le commun des mortels ? Je veux dire hors d'un setup de dev.. Je prendrais le temps de l'ajouter au wiki si vous avez l'info!

Le nouveau feature de diagnostic risque d'aider les gens à fournir de l'information, j'ai pas encore testé remarque mais ca semble bien.. https://www.home-assistant.io/blog/2022/02/02/release-20222/#diagnostics-for-integrations-and-devices

[valleedelisle]

Normalement le maintainer devrait faire au moins une pre-release! Mais y'a oublié apparemment.

Anyways, je merge le PR de @FrancoLoco et je vais faire un release ou pre-release après.

Sinon tu peux toujours aller dans le code et appliquer le commit, c'est pas trop compliqué quand c'est des p'tit commits.

[FrancoLoco]

@arsenicks et @skimaniac avez-vous réussi à updater et faire marcher le tout avec le nouveau pre-release? moi oui mais on dirait que je tombe jamais en catégorie medium.. vous ca marche ?

[arsenicks]

J'update à l'instant je te dis ca.

[arsenicks]

Immédiatement après l'update j'ai le même problème mais le truc d'énergie est pas trop rapide à s,ajuste aux changements si je me fie à mes expériences passé :P Peut être qu'on pourra voir correctement demain ou dans quelques heures ?

Remarque j'ai pas encore atteint le 40 selon mon daily low..

Bizzare d'habitude ca fait longtemps que je l'ai pogné à midi.. Quoi qu'il fait plus chaud..

[FrancoLoco]

ouin ce que je me disais aussi... mais je trouve ca bizarre quand même...

[valleedelisle]

On devrait jaser du problème de comptabilisation sur #80, je crois que cette issue ici est regled.

[arsenicks]

Merci pour le rappel à l'ordre :)

[FrancoLoco]

En fait j'en jasais ici, car je pensais que c'était possiblement un side effect du passage à 2022.2... mais là je vois que mes medium viennent d'embarquer.. je crois qu'il s'est juste passé quelque chose à cause des updates et reboot et tout, mais je crois que ca va tenir maintenant...

[skimaniac]

@arsenicks et @skimaniac avez-vous réussi à updater et faire marcher le tout avec le nouveau pre-release? moi oui mais on dirait que je tombe jamais en catégorie medium.. vous ca marche ?

Je suis pas encore passé en haut de 40kWh ;) je surveille ça.

[arsenicks]

Ca semble clean de mon coté. Je soupconne que les totaux sont fucked up à cause de mon update de HA avant que le pre release de hilo soit dispo mais autrement ca semble fonctionné.

[FrancoLoco]

Oui moi aussi je crois que c’est clean et que ça continuera de bien aller maintenant

[skimaniac]

même chose ici, je vois la différence entre ce matin, avant l'update et après l'update.

[valleedelisle]

@FrancoLoco Depuis que j'ai activé le net_consumption, j'ai de la consommation négative sur mes devices:

[valleedelisle]

Hier aussi j'en avais, mais je croyais que c'était suite au changement + restart + upgrade, etc mais aujourd'hui j'ai rien touché.

Est-ce que tu as le même problème?

[arsenicks]

Tout semble clean ici ce matin!

[valleedelisle]

Tout semble clean ici ce matin!

Avais tu activer le net_consumption?

[arsenicks]

Hmm j'ai la dernière version de l'intégration, j'ai du ratté le bout du net_consumption j'ai rien fait de spécial alors j'imagine que non désolé :P

[valleedelisle]

C'est parfait, merci pour l'info!

@FrancoLoco Est-ce que tu as le meme probleme que moi avec tes devices? Si oui, j'imagine que net_consumption fait du sens sur l'unknown source tracker mais pas sur les autres?

[FrancoLoco]

Non aucun problème ici tout est clean.. mais si tu préfères juste unknown pas de trouble