kamaradclimber / vigieau

A custom integration for home-assistant on the Vigieau dataset
Apache License 2.0
18 stars 3 forks source link

Some sensors report unknown state (but correct attribute) #68

Closed Kelesis closed 3 weeks ago

Kelesis commented 1 month ago

Hi @kamaradclimber ! Thanks for your work on this component.

In my place, alert level has been on "vigilance" for months, and all sensors were reporting "No restriction". Since a few days however, alert level is still on "vigilance", but many sensors are reporting "unknown". In fact if I look attribute for each, they all report the same value (in french) "Sensibiliser le grand public et les collectivités à l’usage économe de l’eau." (Sometimes with a dot at the end, sometimes not 🤷)

This picture might help :

Screenshot_20240810-103725

You can see that it's working for 4 of them. For the others here are exemple of sensor attributes : (Carefully copy pasted)

I can list each of them if you need. (Looking at the code, I can see you are facing some rigor/strictness issues with the consistency of the labels having sometimes extra characters 😅 But I don't know if this issue is related to that.)

Anyway again thanks for your work!

PS : In the main description of this project, it's mentionned that dataset is Propluvia, but it's not anymore, it's now directly VigiEau. (Just follow the links)

Location : Village Center 34110 Mireval (INSEE 34159)

Integration version: 0.5.5 Home-Assistant version: 2024.8.0

yepp31 commented 1 month ago

bonjour @kamaradclimber , j'habite également dans le département 34 et j'ai exactement les mêmes erreurs ... bravo pour ton intégration. image

kamaradclimber commented 1 month ago

Thanks for the reports, the issue indeed come from restriction titles that are not matched by any of the regexp. I'll publish an update this weekend.

On Tue, Aug 13, 2024, at 15:24, yepp31 wrote:

bonjour @kamaradclimber https://github.com/kamaradclimber , j'habite également dans le département 34 et j'ai exactement les mêmes erreurs ... bravo pour ton intégration. image.png (view on web) https://github.com/user-attachments/assets/e0f56891-ec67-4560-bf16-3d4b0c14387b

— Reply to this email directly, view it on GitHub https://github.com/kamaradclimber/vigieau/issues/68#issuecomment-2286248247, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD254IPNCIMLQLWY2JSCELZRICKTAVCNFSM6AAAAABMJVVEICVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWGI2DQMRUG4. You are receiving this because you were mentioned.Message ID: @.***>

jpmartin71 commented 4 weeks ago

bonjour @kamaradclimber , j'ai la même anomalie depuis plusieurs semaines. image J'ai trouvé ce post sur le site https://www.data.gouv.fr/fr/reuses/integration-vigieau-pour-home-assistant/ C'est peu être une piste. image

Integration version: 0.5.5 Home-Assistant version: 2024.1.2 (docker) Interface utilisateur 20240104.0

En tout cas merci pour ton intégration très utile pour moi qui est dans le var.

kamaradclimber commented 3 weeks ago

https://github.com/kamaradclimber/vigieau/releases/tag/0.5.6 should fix the issue.

@jpmartin71, la mention de propluvia était purement attributive car l'intégration utilise depuis le début l'api de vigieau. J'ai mis à jour le readme.

Kelesis commented 3 weeks ago

Hello @kamaradclimber Je viens de tester avec le tag 0.5.6 mais j'obtiens le même résultat qu'avant 😅 J'ai bien vérifié le tag actuel dans HACS, et redémarré home assistant.

Screenshot_20240816-191152

kamaradclimber commented 3 weeks ago

Bien vu, https://github.com/kamaradclimber/vigieau/releases/tag/0.5.8 corrige le problème. J'avais supposé qu'il s'agissait de restriction non supportées mais le problème venait du niveau de restriction. Les catégories vont désormais apparaitrent en "Sensibilisation".

Kelesis commented 3 weeks ago

Perfect 👌 Je valide le fix et même le test d'intégration ! Encore merci pour ce travail très utile

Screenshot_20240816-193757