Open rbalmes opened 1 year ago
Bug fix concernant les données envoyés:
Dans le code initiale on n'envoit pas les données estimées En realité il faudrait prendre en compte l'index du premier jour estimé, il faut prendre en compte l'index 34235 dans le cas ci-dessous.
Voir ci-dessous: Veolia le 27/09 à relevé le compteur -> 33729 Veolia le 28/09 à relevé le compteur -> 34235; Veolia a identifié que 506L ont été consommé le jour précédent Veolia le 29/09 à relevé le compteur -> 34235; Il n'y a pas eu de récupération de l'index du compteur, on ne sait pas la consomation du 28/09, c'est pour cette raison que la consomation est estimé Veolia le 30/09 à relevé le compteur -> 35398, il a donc estimé une consomation moyenne sur les 2 jours précédents
Date de relevé;Index relevé (litres);Consommation du jour (litres);Index Mesuré /Estimé 27/09/2023;33729;506;M 28/09/2023;34235;581;E 29/09/2023;34235;582;E 30/09/2023;35398;823;M
Pour simplier et uniquement traiter les lignes "Mesuré", J'ai considéré que l'index etait = la mesure de l'index en début de journée + la consomation de la journée
A terme je pense qu'il faut prendre en compte les valeurs estimées pour éviter des "trous"
Gestion des données estimés
J'ai constaté dans le passé que les données estimés sont farfelus. S'ils sont à prendre en compte, je ne le ferai qu'après avoir reçu une donnée ultérieure permettant de vérifier que les valeurs soient bien en ordre croissant.
Je n'ai pas encore regardé le code car j'ai qqs truc "sur le feu".
J'ai formatté le code, corrigé les "problèmes" identifiés par les analyses statiques, corrigé qqs typos.
Ce qui ne dépend pas de HA devrait bouger dans un premier temps dans Injector ou les fonctions devraient retourner des tableaux utilisées ensuite par les classes enfants.
J'espère que je n'ai pas introduit de bogue.
Je pense également qu'il n'y a pas de dépendence avec spook puisqu'on appelle le web service de HA. Spook propose "juste" un service utlisable dans les automatisations ou interactivement.
@mdeweerd Quand est-ce que tu as prévu d'ajouter ce commit dans le default branch?
Je veux vérifier/faire que ce soit assez général et "robuste" - quand j'ai regardé au moment de la soumission, je trouvais qu'il fallait encore le travailler. Une fois que c'est dans main, il faut que cela fonctionne et c'est plus compliqué d'apporter des modifications.
Generaliser, c'est entreou outre avoir une fonction ou l'on ajoute l'historique à une "variable"/"un état" au choix en fournissant le tableau des paires date/valeur entre outre. Ce qui est valide pour l'eau l'est aussi pour le gaz, etc.
Ajout pour supporter : unicontrat (www.service.eau.veolia.fr) Ajout du chargement des données historiques de veolia