Tuya : Intégration chauffage et climatiseur

Demande de fonctionnalités
41

Bonjour @pierre-gilles ,

Je pense être prêt pour une review de cette PR qui met à jour l’intégration Tuya en préparant la base pour la nouvelle connexion et la détection locale :

Merci à toi par avance.

EDIT : Après un recheck complet, j’ai vu que la PR aurait dû etre découpée en plus d’éléments. J’ai donc préféré la mettre en standby. Je viens de la réouvrir. @pierre-gilles elle est prête à review.

EDIT 2 : Pierre-Gilles, pour tester toutes les PR Tuya, je peux tout à fait partager mon compte si tu le souhaites.

2 « J'aime »
42

Une fois la précédente valide, je mettrais à jour les PR suivantes :

  • Ajout du protocol 3.5 nécessaire pour les équipements récents. Gros avantage par rapport à HA, ceci fera partie intégrante de Gladys alors que pour HA il faut passer par une intégration externe HACS :
  • Mise en place de la base de mapping des appareils cloud vs local. J’ai pris les Smart Socket pour travailler cette base qui permettra d’ajouter plus simplement les futurs appareils / fonctionnalités :

Elles sont également prêtes et n’attendent que la 1ère ^^

Ensuite j’ai séparé une PR pour l’intégration des création automatiques des issues github.

1 « J'aime »
43

Merci beaucoup pour les PRs, je te tiens au courant quand j’ai pu regarder !

44

@Papashultz et @GBoulvin, je viens de relancer une construction sur la meme image de test, je serais très interessé de votre nouveau test et de vos retours.

Pour info @GBoulvin j’ai ajouté la prise en charge de ces issues :

Je pense et j’espere que la derniere gère également le fil pilote de @Papashultz

Et pour info, voici à quoi ressemble les Issues dorénavant :

45

Nouvelle issue traitée en PR :

  • Appareil :

image
image335×927 22.7 KB

  • Dashboard

image
image592×356 14.1 KB

  • Balayage horizontal :

image
image565×137 6.93 KB

  • Balayage vertical :

image
image585×243 10.4 KB

  • Mode :

image
image585×189 8.46 KB

  • Vitesse de ventilation :

image
image588×266 4.33 KB

1 « J'aime »
46

Hello !

Merci pour cette update !

Pour ma part, le “Smart meter” est toujours indiqué comme “non pris en charge” :

image
image423×872 18.6 KB

Pour les prises LSC, Une des deux (protocole 3.5) est fonctionnelle et proposée en tant que prise en charge (avec sécurité enfant !):

image
image436×881 20.5 KB

Par contre la seconde (protocole 3.4) me retourne une erreur (en local).

image
image422×677 15.8 KB

J’ai créé une issue GitHub à l’aide du bouton et voilà le résultat (ça fait pro !) :

Si je puis aider , n’hésite pas !

Edit : Pour la seconde prise, après re-test le cloud fonctionne mais pas le local.

1 « J'aime »
47

Bonjour @GBoulvin,

Merci pour tes tests

Quelle déception … J’espérais qu’il match du 1er coup celui-ci ^^

48

Re-test avec l’image du jour !

Pour le Smart Meter, les virgules sont manquantes :

image
image442×822 12.5 KB

Mais toutes les fonctionnalités apparaissent correctement :

image
image351×167 3.93 KB

Par contre, les appareils ont disparu du tableau de bord après avoir mis à jour en local (c’était résolu, non ?)

Est-ce que le scan local doit absolument être séparé du scan cloud ou ça pourrait être automatique ?

Parfois, je dois mettre les adresses IP manuellement et je n’arrive pas à déterminer pourquoi ou et pourquoi non…

Et je confirme que les protocoles 3.3, 3.4 et 3.5 sont bien fonctionnels en local !!!

1 « J'aime »
49

Salut @GBoulvin

Désolé pour le contre temps, c’était un test … malheureusement non concluant !! Enfin il y avait un oubli surtout. L’image est mise à jour et en cours de construction => 5 à 20 minutes.

Oui pour le coup là ça peut être normal, j’ai changé des choses dans la feature donc ça à surement cassé l’ancien device (pas de migration automatique). Comme on est en dev, je ne fais pas de migration qui casserait l’ancien device en cours de modification ^^

Oui, c’est volontaire pour l’instant. Le scan UDP local est plus long et dépend du réseau local, donc j’ai préfèré le garder en action manuelle pour ne pas ralentir la découverte cloud.

C’est cohérent avec le fonctionnement UDP local: selon le réseau (AP isolation, VLAN, filtrage broadcast/multicast, device qui broadcast peu), l’IP peut ne pas remonter automatiquement.
Dans ces cas, la saisie manuelle reste nécessaire. Il ne s’agirait pas d’un appareil sur un réseau différent ou qui metterait plus de temps a répondre à ton avis ?

1 « J'aime »
50

Salut @Papashultz et @GBoulvin ,

Dites, @pierre-gilles à commencé la review de la 1ere PR sur Tuya, auriez-vous la possibilité de tester la 1ere PR lorsque celle-ci est validée côté code en partant d’une copie de votre base production pour être certains que ça ne casse rien sur du matériel en fonction ?

J’ai pu tester de mon côté avec 2 prises (seul équipements supportés sur la version en vigueur pour moi), mais ça ne mène pas très loin.

Autres appels à contributeurs si vous aviez la possibilité de tester en tant que possesseur d’appareils Tuya :

L’image de test sera disponible sous docker pull terdious/gladys:tuya-PR1

2 « J'aime »
51

Bonsoir @Terdious
J’arrive après la bagarre, mais j’ai testé ton image. Voici ce que j’ai testé comme appareils :

image
image1200×472 28.6 KB
. Tous fonctionnent, il n’y en a qu’un qui ne fonctionne pas entièrement, c’est l’ampoule de couleur. On voit déjà que sur le dashboard les couleurs sont sombres,
image
image354×242 11.7 KB
. C’est parce que la lampe n’est pas bien reconnue, il manque des fonctions telles que : mode blanc, mode couleur, mode scénario, qui sont dans l’app Tuya. Mais le problème était déjà présent avant que tu ne reprennes le projet. J’espère que ça peut t’aider, merci pour ton travail. Bonne soirée

2 « J'aime »
52

Salut @Psoy !!

Merci beaucoup !! Tu n’arrives pas du tout après la bagarre, au contraire !!
Super retour rassurant !!

Si tu veux tester plus, tu peux tester dès maintenant docker pull terdious/gladys:tuya-test, elle devrait te permettre de créer une issue github pour que j’ajoute la reconnaissance complete des appareils où il manque des choses. Si elle sont présentent en issue, je peux travailler dessus de temps en temps quand j’ai un peu de temps ^^

1 « J'aime »
53

Bonjour @Terdious
J’ai testé avec cette image docker pull terdious/gladys:tuya-PR1
Je vais installer docker pull terdious/gladys:tuya-test, mais pour l’issue GitHub, je reviendrai vers toi, car là je ne maîtrise pas :wink:

1 « J'aime »
54

Ca se fait tout seul tu verras !! Et justement intéressé d’avoir ton retour sur la compréhension du petit fil à suivre ^^ :wink:

Le but lors de ce développement était de poursuivre ce qui est fait sur quelques autres intégrations mais en guidant encore plus !! Et sans créer de doublons de demande

3 « J'aime »
55

Merci pour ton retour @Psoy !

Si quelqu’un d’autre peut tester l’image, ça serait top ! Ensuite, c’est bon pour moi pour déployer cette première PR.

Merci @Terdious pour le développement :raising_hands:

2 « J'aime »
56

J’ai peur que non, il va être difficile d’avancer malheureusement.

Pour ma part, j’avais quand meme un vrai besoin. Je me suis donc motivé et meme si beaucoup plus compliqué pour les climatisations que ce que j’avais dev ici, c’est entierement installé dans HA. Et ca fonctionne très bien. Ca m’a couté de devoir refaire toutes mes scenes pro dans HA, mais ca à le mérite de fonctionner (et de me faire faire des économies depuis 2 semaines).

Etant depuis 2 semaines dans une période fast coté boulot, et maintenant entrant dans la période des extérieurs (travaux / entretriens), je préfèrais prévenir ici que je n’aurais plus vraiment de dispos, comme chaque année, jusqu’en octobre.

Pour ma part je n’aurais plus de besoin urgent de cette intégration, je tenterais de suivre quand meme si il y a une évolution et si je trouve quelque dispo repo à ce moment je ferais ce qu’il faut pour ceux qui en ont le besoin.

Le controle d’activité automatique sur la PR github fera le job pour les femer sinon.

A bientot ! Et bon printemps/été à tous.

1 « J'aime »
57

Merci pour ton retour et ton implication @Terdious :slightly_smiling_face:

Je pense que les beaux jours sont de retour et que l’activité baisse naturellement, tu n’es pas le seul à être plus occupé en ce moment :grinning_face_with_smiling_eyes:

Je le vois aussi sur les réseaux, j’ai moins de vues qu’en début d’année !

J’espère qu’on aura d’autres testeurs :crossed_fingers: