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

check_tarif: Unable to find state for sensor.hilo_energy_total_daily_low #15

Closed
arsenicks opened this issue Dec 24, 2021 · 14 comments
Closed

Comments

@arsenicks
Copy link
Collaborator

arsenicks commented Dec 24, 2021

Version of the custom_component

Latest 23dec. 20h50: 79f8ca2

Configuration

J'ai ajouté 'utility_meter:' à mon configuration.yaml
J'ai activé le "Generate energy meters" dans l'intégration, comme décrit dans le bug #12 il se décoche, alors pour tester j'ai modifié "DEFAULT_GENERATE_ENERGY_METERS = False" dans custom_components/hilo/const.py pour le mettre à true.

Describe the bug

Même bug à la base que #10

sensor.hilo_energy_meter00* sont généré mais pas de sensor.hilo_energy_total_daily*

Debug log

J'ai activé le debug mais je ne trouve rien de plus utile que ceci comme erreur/warning. Laisse moi savoir si tu saurais quoi chercher exactement. Je peux t'envoyer un grep de tout les log qui contienne hilo mais je ne veux pas le poster publique..

2021-12-23 20:39:37 WARNING (MainThread) [custom_components.hilo] check_tarif: Unable to find state for sensor.hilo_energy_total_daily_low
2021-12-23 20:39:37 DEBUG (MainThread) [custom_components.hilo] Finished fetching [email protected] data in 0.983 seconds (success: True)

@valleedelisle
Copy link
Contributor

L'erreur de check_tarif devrait avoir été adressé avec 2a32446.

Je suis sur le point de soumettre une patch pour les settings qui ne "savent" pas.

valleedelisle added a commit that referenced this issue Dec 24, 2021
Apparently, the suggested_value is the way to go to show the current
status of the various options.

Fixes #12,#15
valleedelisle added a commit that referenced this issue Dec 24, 2021
Apparently, the suggested_value is the way to go to show the current
status of the various options.

Fixes #12,#15
@FrancoLoco
Copy link
Collaborator

J’ai le même problème et même avec toutes les patches ça perdure..

@arsenicks
Copy link
Collaborator Author

Je viens de mettre à jour avec la dernière version, je vais laisser rouler quelques temps voir si l'erreur réapparait.

@valleedelisle Normalement avec l'option Generate energy meter activé et le configuration.yaml modifié pour inclure "utility_meter:" l'entité sensor.hilo_energy_total_daily_low devrait toujours être généré ? Car elle n'est jamais apparu ici.

@FrancoLoco
Copy link
Collaborator

En regardant un peu le code je croyais avoir trouvé quelque chose pour me débloquer, on dirait qu'il s'attend à ce que le compteur s'appelle "SmartEnergyMeter" et non pas "meter_00".. mais même en changeant, ca ne semble toujours pas marcher..

@valleedelisle
Copy link
Contributor

Je viens de mettre à jour avec la dernière version, je vais laisser rouler quelques temps voir si l'erreur réapparait.

@valleedelisle Normalement avec l'option Generate energy meter activé et le configuration.yaml modifié pour inclure "utility_meter:" l'entité sensor.hilo_energy_total_daily_low devrait toujours être généré ? Car elle n'est jamais apparu ici.

Oui elle devrait toujours être générée. Si elle n'est pas générée, quelques chose s'est produit. Si tu veux bien activer le debug mode et m'envoyer les logs home assistant, ça pourrait m'aider.

@valleedelisle
Copy link
Contributor

En regardant un peu le code je croyais avoir trouvé quelque chose pour me débloquer, on dirait qu'il s'attend à ce que le compteur s'appelle "SmartEnergyMeter" et non pas "meter_00".. mais même en changeant, ca ne semble toujours pas marcher..

Je ne vois pas comment vous pouvez avoir un meter00? Le SmartEnergyMeter est mappé sur hilo_energy_total.

@FrancoLoco
Copy link
Collaborator

Dans nos devices le compteur s'appelle Meter00 et non pas SmartEnergyMeter

@FrancoLoco
Copy link
Collaborator

All right je crois avoir réglé en modifiant les 2 bouts de codes où on vérifiait pour "SmartEnergyMeter" et j'ai mis "Meter00" en plus pour les mêmes IF et ca semble marcher là.. 🤞

@arsenicks
Copy link
Collaborator Author

arsenicks commented Dec 24, 2021

Je ne vois pas comment vous pouvez avoir un meter00? Le SmartEnergyMeter est mappé sur hilo_energy_total.

Je te confirme que c'est meter00 ici aussi, on a peut être un autre modèle de coordo zigbee ?Je vois "modelNumber': 'HT402'" dans les logs.
image

Si tu me fourni un courriel perso je pourrais t'envoyer le debug mais il y a un peu d'info perso alors je ne voudrais pas le mettre ici en intégralité.. Tu peux m'écrire à [email protected] si tu ne veux pas la rendre publique.

Sinon voici un bout qui te démontre la création des meter00:

2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Returned data: [{'onlineStatus': 'Online', 'lastStatusTimeUtc': '2021-12-23T17:23:55Z', 'zigBeePairingActivated': False, 'dsn': 'AC000W008487938', 'installationCode': 'AC27EFFB5A86AC7A', 'sepMac': 'XXXXXXXXXXXXXXX', 'firmwareVersion': '2.2.1', 'localIp': None, 'zigBeeChannel': 24}]
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Sensor device: Meter00
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Climate device: Thermostat XXXXXXXXXXXXXXX
2021-12-23 20:39:32 DEBUG (SyncWorker_3) [paho.mqtt.client] Sending SUBSCRIBE (d0, m281) [(b'zigbee2mqtt/bridge/state', 0)]
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Climate device: Thermostat chambre XXXXXXXXXXXXXXX
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Climate device: Thermostat Salle de bain SS
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Climate device: Thermostat salon bas
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Climate device: Thermostat bunker
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Climate device: Thermostat chambre XXXXXXXXXXXXXXX
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Climate device: Thermostat chambre XXXXXXXXXXXXXXX
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Climate device: Thermostat sale de bain haut
2021-12-23 20:39:32 DEBUG (MainThread) [pyhilo] Setting up Sensor device: Hilo Gateway

@valleedelisle
Copy link
Contributor

Je t'ai envoyé un courriel pour les logs, c'est peut-être atterri dans ton spam. Mais j'pense bien avoir une bonne idée de comment fixer ça. Je vais harmoniser en utilisant le model number, soit SmartMeter ou HT402 et je crois que je vais changer le nom du device pour SmartEnergyMeter au lieu de meter00 dans la librairie pyhilo. Vous en pensez quoi?

@arsenicks
Copy link
Collaborator Author

arsenicks commented Dec 24, 2021

Yes, c'est recu, je suis entrain de t'envoyer le log.

Ca me semble clean comme solution!

Merci!

@valleedelisle
Copy link
Contributor

Après analyse des logs, voici ton SmartMeter

  {
    "id": 347721,
    "assetId": "xxx",
    "identifier": "xxx-smart_meter",
    "gatewayId": xxx,
    "gatewayExternalId": "xxx",
    "name": "Meter00",
    "type": "Meter",
    "groupId": "None",
    "category": "Other",
    "icon": "None",
    "loadConnected": "None",
    "modelNumber": "None",
    "locationId": xxx,
    "parameters": "None",
    "externalGroup": "SmartMeter",
    "provider": 1,
    "providerData": "None",
    "disconnected": "False",
    "supportedAttributes": "Power",
    "settableAttributes": "Power",
    "supportedParameters": "None"
  }

Y'a l'air d'avoir un bug avec Hilo, j'essaierais de faire un factory reset sur ton gateway pour voir si ça ne rêgle pas le problème. Sinon je contacterais Hilo pour ton problème de défi, c'est pas normal. En attendant, je vais essayé de trapper l'externalgroup au lieu d'y aller par nom. Les HT402 sont tes Thermostat.

Une chose est sûre, c'est qu'en vérifiant les ID, on voit qu'il y a des readings qui rentrent pour ton SmartMeter, donc au moins les données sont là.

Je vais pousser une patch au courant de la journée pour ça.

@Fox350
Copy link

Fox350 commented Dec 24, 2021 via email

@FrancoLoco
Copy link
Collaborator

Marche bien good job

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants