gduteil / cozytouch

Atlantic Cozytouch integration for Home Assistant
22 stars 0 forks source link

KELUD 1750W (Blanc) - Contrôle de la température impossible #14

Closed AuroreVgn closed 4 months ago

AuroreVgn commented 4 months ago

Bonjour,

J'utilise depuis quelque mois cette intégration pour contrôler dans HA mon KELUD 1750W (Blanc). Elle fonctionnait à merveille.

Je l'utilise notamment pour faire une scène qui l'allume à 26°C (_number.kelud_1750w_blc_targettemperature) pendant 1 heure (_number.kelud_1750w_blc_override_totaltime) afin de faire sécher les serviettes. Une automatisation permet ensuite de l'éteindre.

Or, depuis hier (lien avec la mise à jour 2023.7.0 ???), seuls l'allumage et l'extinction fonctionnent correctement.

Depuis HA, le contrôle de la température fonctionne aléatoirement que ce soit avec la carte 'climate' ou le slider de _number.kelud_1750w_blc_targettemperature. En effet, le message d'erreur apparait

Erreur

Puis, parfois la température se met à jour dans l'appli CozyTouch et parfois pas. Le même message d'erreur apparait dans la scène qui programme la température et, étonnement, dans l'automatisation qui l'éteint.

A noter, les modifications réalisées depuis l'appli CozyTouch sont prises en compte instantanément dans HA.

Version CozyTouch : 0.80.0 Version HA : Core 2024.7.0 / Supervisor 2024.06.2 / Operating System 12.4 / Frontend 20240703.0

AuroreVgn commented 4 months ago

Re,

J'ai tenté une correction rapide (et probablement pas très propre) : j'ai commenté la ligne 491 time.sleep(1) du fichier _customcomponent > cozytouch > hub.py, redémarré HA et le problème est résolu.

gduteil commented 4 months ago

Bonsoir,

Certaines capabilities prennent un peu de temps à être activées, et dans certains cas où il faut changer plusieurs valeurs successivement il faut attendre que les précedentes soient effectives pour que les suivantes soient acceptées, d'où cette partie du code qui interroge l'API pour savoir si la requête précedente est terminée avant de lancer la suivante. Il me semble que le seul cas où c'est nécessaire est dans le cas d'un "override" (délégation) d'une consigne en étant en mode programmation, les sèche-serviettes n'ont pas de mode programmation il me semble, donc ça ne devrait pas poser de problème.

La dernière MAJ de HA n'a plus l'air d'accepter cette attente avec le time.sleep(1), je vais regarder ce week-end pour remplacer par une autre solution.

gduteil commented 4 months ago

Au final c'était pas grand chose à modifier, je viens de publier une 0.9.0, tu peux mettre à jour pour voir si ça fonctionne ?

AuroreVgn commented 4 months ago

Bonjour,

Merci pour ton retour. Le KELUD 1750W (Blanc) bien un mode programmation et donc un mode override que j'utilisais pour ma fameuse automatisation de séchage. Je viens de faire la mise à jour 0.9.0 ainsi que de HA Core 2024.7.1 et cela fonctionne parfaitement ! Merci beaucoup !

gduteil commented 4 months ago

Parfait !