[Virtual] Infos remontent en double depuis MàJ

Re,

Bon après un petit dif entre la version en place sur mon systeme hier et la nouvelle version il n’y a pas des tonnes de modif mais il y en a une qui attire le regard. Cette ligne a été ajoutée dans la méthode « execute » pour les commandes de type info…

$this->event($result);

ça ne doit pas être une coïncidence.

Quelqu’un de l’équipe peut regarder ?

1 « J'aime »

J’étais dessus également, c’est effectivement cette ligne 318 de ce commit qui est la cause je pense:

je ne suis pas sur de pourquoi elle a été ajoutée, j’imagine que dans certain cas si il y avait une expression à évaluer ce n’était pas la bonne valeur qui était envoyé la première fois?

ok, @Loic a changé la version en beta il y a 4h en commentant cette ligne, ca va être fixé donc

Salut, je confirme, j’ai essayé avec seulement la MaJ du plugin, et pareil.

Je viens de faire le test sans cette ligne et comme espéré il n’y a plus qu’un seul déclenchement.

Maintenant reste à savoir pourquoi cette ligne a été ajoutée et dans quel cas elle est utile…

je viens de faire un petit tour sur le github la ligne à été mise en commentaire dans les sources il y a 4h donc Loic travaille sur le sujet :wink:

grillé par @Mips

1 « J'aime »

Je vient de testé en version beta, sa résout bien le problème, en attente de la version finale.
Merci a tous.
Dois-je mettre le Topic en résolu ?

Je propose d’attendre pour mettre le problème en résolu.
La version n’est pas dispo.
Merci

1 « J'aime »

J’ai testé également. A priori ça fonctionne. Pas idée d’autres régressions éventuelles que ceci pourrait engendrer en revanche.

Bonjour

Y’a un petit détail qui me dérange, ce sont ces 3 lignes :

Changelog Virtuel

IMPORTANT

Pour rappel s’il n’y a pas d’information sur la mise à jour, c’est que celle-ci concerne uniquement de la mise à jour de documentation, de traduction ou de texte

Or, il n’y a pas eu d’information sur la mise à jour de mi-mai et on a bien vu que du code avait été modifié… Un oubli ???

Merci quand même au développeur qui fait vraiment un super boulot !

Bonsoir,

je viens de découvrir une nouvelle « régression » dans le plugin virtal, en plus de faire marche arrière sur le problème rencontré précédemment une autre modification à été faite dans la dernière mise à jour.

$virtualCmd->event($result);

remplacé par

$eqLogic->checkAndUpdateCmd($virtualCmd,$result);

Et tout mon système d’interrupteurs qui utilise des virtuels ne fonctionne plus et évidement c’est en rentant ce soir que je m’en rend compte. Heureusement que j’avais vu ce matin cette modification. Mais sans en mesurer les conséquences.

Un retour à la version précédente de cette ligne et c’est repartit.

Bon on verra demain si j’arrive à expliquer mon nouveau problème ou alors si l’équipe explique la semaine prochaine cette nouvelle modification…

ken@vo
Phil

Finalement j’ai pris 5 minutes pour tenter de comprendre et le checkAndUpdateCmd ça doit être pour vérifier le comportement des commandes info suivant le paramétrage « Gestion de la répétition des valeurs ». Dans mon cas mes commandes d’interrupteurs ne sont pas configurées en « toujours répéter » et entre deux appuis elles ne changent par obligatoirement de valeur. Si j’ai bien compris jusqu’à la dernière modification, le plugin virtuel ne tenait pas compte de ce réglage ce qui vient d’être fait.

J’ai fait le test sur une commande et ça fonctionne.

On va dire que je suis bon pour corriger toutes mes commandes de ce type pour les passer en « toujours répéter ».

Pour ce soir je laisse l’ancienne ligne de code on verra demain mais si vous rencontrez le même problème vous pouvez corriger vos commandes.

Ken@vo
Phil

1 « J'aime »

Bonjour

Pour info, une nouvelle version du plugin Virtual est dispo.

Merci au développeur !

Bonjour, @Pacha95

Si ton message s’adresses à moi au sujet de mes deux derniers message, c’est avec la dernière version justement que mon « problème » arrive

ken@vo
Phil

Bonjour, Je rebondit sur ton Post @Phil56, J’avoue que je n’est jamais trop compris la différence entre « Automatique » et « toujours répéter ». A part le fait qu’on pouvait déduire dans les log que c’était une répétition ou pas.
Avant les MàJ de mémoire quand l’infos était en « Automatique » cela revenait a l’option « Toujours répéter » avec juste l’ajout (répétition) dans le log,

Automatique avant MaJ :

[2020-05-17 10:09:50][INFO] : Exécution de la commande [Maison][TEST3][ON] avec les paramètres {"utid":"xxxxxxxxxxxxx"}
[2020-05-17 10:09:50][INFO] : Evènement sur la commande [Maison][TEST3][ETAT] valeur : 1 (répétition)

Et effectivement maintenant après la MàJ, L’option « Automatique » agit comme l’option « jamais répéter ».

Automatique après MàJ :

[2020-05-17 10:14:27][INFO] : Exécution de la commande [Maison][TEST3][ON] avec les paramètres {"utid":"xxxxxxxxxxxxx"}

@Phpvarious

De ce que j’ai compris suivant si ta commande est binaire ou numérique le comportement d’automatique n’est pas le même.

Donc si en automatique

  • numérique = répétition des mêmes valeurs
  • binaire = pas de répétition si mêmes valeurs

ken@vo
Phil

Ah Oui effectivement je vient de tester.
Donc pour résumé,

En Automatique :

  • numérique = répétition des mêmes valeurs.
  • binaire = pas de répétition si mêmes valeurs.
  • Plus de possibilité de voir dans les log que c’est une répétition.
1 « J'aime »

pour ton dernier point je pense que c’est parce que justement avec la dernière modification il n’y plus répétition si pas nécessaire…

Donc autant enlever l’option Automatique, et avoir que les autres options disponible, car si en fonction de numérique ou binaire on a pas le même comportement en automatique, sa me semble un peu compliqué l’histoire. :grin:

@mchacher, si tu passes par ici :wink: et que tu as 5mn pour poster sur GitHub. Merci.
J’avoue que je suis un peu nouveau dans le domaine (découverte de Raspberry, Jeedom, Linux… y’a pas plus de 6mois :grin:).
Donc je sais pas trop comment fonctionne ce petit monde de développeurs, est-ce qu’ils ont le temps de vérifier les problèmes remontés dans la communauté ou si vaut mieux posté sur GiHub.
En tout cas merci a eux.

Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.