-
Notifications
You must be signed in to change notification settings - Fork 15
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
Conversation
…ections in device info
…ns that do not have by-id mechanism
Update setup and ConfigEntry
refactor status register handling fix absolute import remove unecessary waiting (now handled by async_setup) change serial function to property
fix debug messages number of args fix log flood of availability infos
change device class for consistency add custom on/off icons
Après une longue pause sur HA, je vais refais une petite passe pour finir ce qui doit l'être. Sacré MR. Pour être honnête, pas le temps de tout relire... Je vais merge et tester le code voir si tout fonctionne :D (je suis encore en mode historique pas d'inquiétude) Merci pour le travail 👍 |
Dites moi si tout fonctionne comme il faut: |
Si la beta 4 est ok pour vous, elle deviendra la 3 definitive. |
Linky en mode Standard ici, la mise à jour fonctionne bien de mon côté :
Edit: j'ai rencontré un petit bug avec une icone manquante. Je fais une PR. ➡️ #45 |
Plusieurs mois de fonctionnement sans détecter de problème en mode standard (quelques erreur de checksum mais rien d'alarmant), donc sur cette partie tout a l'air de fonctionner 🚀
Hélas c'est le prix à payer pour simplifier la classification 😉
Pour les appareils vides, il me semble que nettoyer les entités qui leurs sont rattachés devrait suffire à les faire disparaitre (c'est HA qui génère et détruit les appareils en fonctions des données exposés par les entités) ? Peut-être encore quelques recherches à faire à ce sujet. PS: |
Ces appareils vides restent un mystère pour moi. HA est sensé les nettoyer de lui même quand plus aucune entité ne les déclare. |
Résolution de bogues :
by-id
lorsque disponible (2 appareil détecté #27)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 lesCR
par desLF
)Factorisation :
Changements :
BinarySensorDeviceClass
à ajuster)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 😅