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

Fix duplicate devices, empty lines and refactor entities #32

Open
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

tomleglaunec
Copy link
Contributor

Résolution de bogues :

  • Utilisation des ports séries via leur chemin persistant by-id lorsque disponible (2 appareil détecté #27)
  • Vérification de la présence du compteur pour éviter la création de doublons vides (2 appareil détecté #27)
  • Omission des lectures vides (devrait supprimer les logs d'erreurs Failed to parse: b'' Error in historic mode: 'b' #26, même si la racine du problème est l'interface série qui remplacerait les CR par des LF)

Factorisation :

  • Création d'une entité abstraite dont dérivent les entités de l'intégration
  • Factorisation des méthodes de mise à jour des données
  • Création de classes de capteurs spécifiques (Puissance instantanée, Tension, etc.)

Changements :

  • Passage de certains capteurs liés au registre de statut en capteurs binaires (BinarySensorDeviceClass à ajuster)
  • BREAKING v3.0.0beta3 : Changement de la méthode d'attribution des id uniques pour les entités de registre de statut (apparaitront comme de nouveaux capteurs pour ceux utilisant déjà la beta3, les anciens ne seront plus fournis)

IMPORTANT : N'ayant qu'un compteur linky en mode standard, je n'ai pas pu vérifier que les modifications ne cassent pas le mode historique, si quelqu'un pouvait m'envoyer une capture de quelques trames historiques, je pourrai tester l'intégration

Les PR ne devraient pas être aussi fourre-tout, au risque de les rendre indigeste, en espérant que la relecture ne sera pas trop dure 😅

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

Successfully merging this pull request may close these issues.

1 participant