[Store] Toggle

[merguez07]

Bonjour,
Avec l'accord d'Influman je viens de monter d'indice le plugin Toggle afin que chacun puisse définir les valeurs associées à l'état 0 et l'état 1 d'un actionneur (initialement on imposait le 0 et 100).

Pour rappel le Toggle vous permet de commander par simple sollicitation, 2 états d'un actionneur :
- on sollicite le Toggle, il active l'état 1 de actionneur associé
- on resollicite le Toggle , il active l'état 0 de l'actionneur associé
- et on recommence,

Perso je trouve ce plugin très pratique pour commander à la voix via GH et IFTTT mes actionneurs bistables.
Je dis "ok Google, lumière" la lumière s'allume, je redis "ok google, lumière" la lumière s’éteint.
Plus besoin de 2 applets IFTTT, 1 seul fait le job .
plus besoin de deux boutons sur vos télécommandes pour mettre en marche et arrêter un actionneur

enjoy

[anne-marie]

Pas mal! Effectivement, ça peut faire gagner du temps.

[Olivepoilue]

très pratique !
mais est-ce que ce serait facile d'étendre ce script en lui faisant gérer x états au lieu de seulement 2 ?

ça pourrait être utile pour un interrupteur qui permettrait de gérer plusieurs niveaux d'intensité lumineuse. J'ai, pour ma part, créer 5 règles par interrupteur, et ce pour chaque chambre ... ça fait un paquet de règles que le script pourrait faire éviter

[influman]

c'est pas ce que fait le plugin "incrémentation de la luminosité" ? je me trompe sans doute

[Olivepoilue]

merci Influman, je n'avais pas vu ce plugin
mais chui pas sûr sûr... c'est vrai que le nom est explicite mais dans la description, il est évoqué la gestion d'une telecommande 4 boutons. On dirait que chaque bouton a 1 niveau de luminosité distinct
ba je te testerai tout de même pour voir

[influman]

Ah oui t’as sans doute raison.
Tu pourras utiliser le plugin ToggleChain quand il sera validé sur le Store...

[Olivepoilue]

super, merci bcp Influman, ça c'est de la rapidité !

[Olivepoilue]

It works !! Great job

[Olivepoilue]

en jouant avec togglechain, je me suis mis à rêver d'un plugin togglescene car j'y voyais une utilité sur un cas bien précis, ma fille et ses "ambiances" lumineuses (truc de fille) dans sa chambre. Elle a des combinaisons entre sa zipatobulb et son ruban mais difficilement gérables avec 2 togglechain synchronisés ...
j'ai testé sur togglechain sans trop y croire mais si, ça gère bien les scenes, il suffit de nommer les états que l'on souhaite par leurs valeurs ! Trop bon ce plugin !

[jbourdin]

Petite suggestion sur la lisibilité du code PHP pour Toggle.
On évite les imbrications de if et les else si on sort dès qu'on a terminé (pas de périphérique, on sort, on encapsule pas le reste dans un if).
Également, on utilise les arguments de getArg pour alléger le code (valeur par défaut et mandatory).
Question, $periphId != 'plugin.parameters.device_id', ça sert à quoi ? (je l'ai laissé dans le code)

Ah, et j'ai retiré le ?> en fin de fichier, c'est inutile et ça peut faire des retours non prévus si des caractères sont après. (cf https://www.php-fig.org/psr/psr-2/#22-files)

Code : Tout sélectionner

<?php

// recuperation des infos depuis la requete
$periphId = getArg("periph", true);
$val1     = getArg("val1", false, 0);
$val2     = getArg("val2", false, 100);

// No exec if not for plugin.parameters.device_id
if ($periphId == 'plugin.parameters.device_id') {
    die;
}

$periphinfo  = getValue($periphId);
$periphvalue = $periphinfo['value'];

// set to val2 if already on val1
if ($periphvalue == $val1) {
    setValue($periphId, $val2);
    die;
}

// set to val1 if not on val1
setValue($periphId, $val1);
die;



[influman]

Je ne suis pas du tout programmeur ni spécialiste php, donc c'est clair que rien dans ces plugins n'est conventionnel. Si t'as envie de relire, retoucher, optimiser le code, fais toi plaisir, j'ai une 30aine de plugins dans le store.
Par contre tu gères après la maintenance, la communication, les régressions, sur le forum
avec réactivité et disponibilité

[jbourdin]

J'apporte mes compétences si ça peut aider.
L'idéal pour maintenir tes scripts, c'est de les mettre en open-source, sur github par exemple. On pourra alors facilement te proposer des corrections.

Et c'est en faisant la revue de code qu'on progresse et fait progresser, ce n'est en aucun cas un reproche, juste un exemple de comment faire plus clair.

[influman]

...

[merguez07]

...

lol, je crois que notre barbu est en train de te taper sur le système.

[influman]

...

lol, je crois que notre barbu est en train de te taper sur le système.

C'est pas ça.
C'est juste que dans ma signature il y a un lien vers un github,
et que je me suis justement fendu de tout mettre sur ce truc pour gérer la configuration et partager gratos, même à ceux qui n'ont pas accès au store eedomus.
Donc me demander de faire un github pour qu'on puisse m'aider à m'améliorer en code...bref

Après faire un premier message sur le forum pour donner des leçons de code sur un script de 3 lignes, je trouve cela pas très bien amené, ni très sympa. Au moins certains commencent par des mercis sur les usages que j'ai tenté de leur offrir avec mon temps perso et mes faibles compétences...avant de me suggérer des choses et me conseiller.
Mais on n'a pas tous la même sensibilité dans la vie, certes.

A choisir, je préférerais dicter l'usage souhaité imaginé à un jeune programmeur payé par eedomus...plutôt que de devoir mettre les mains dans le cambouis pour que ça prenne vie rapidement...

[jbourdin]

My bad sur le github, j'ai pas vu (lu) ta signature.

Pour la suggestion de code, ce n'était pas mon premier message, même si je ne m'étais pas présenté avant, et je suis désolé que tu le prennes comme ça. Ayant eu des soucis sur l'exécution du script, je l'ai ouvert pour faire du debug (et aucun bug au final, c'est la chaine action puis règles qui induit de la latence quand le script est instantané). Comme souvent sur du code open que je suis amené à toucher, j'ai fait un retour, ce qui me semble justement le principe de la coopération open source. N'ayant pas de donnée git dans le code, je suis venu sur le forum et j'ai trouvé ce sujet.

Je suis donc désolé de t'avoir froissé, cela ne représente pas du tout l'opinion que j'ai de ton travail. J'ai moi même fait des travaux open et contribué occasionnellement aux travaux des autres, j'ai toujours pris les propositions comme un premier remerciement sur le projet. Si pour toi ce n'était explicite, je le dis plus clairement : merci à toi du temps et des efforts fournis pour aider la communauté, n'hésite pas à me solliciter si tu as besoin d'aide, je serai ravi d'aider dans la limite de mon temps disponible.

P.S- la version de github n'incluait pas les changements de la v1.1, je t'ai fait une pull-request incluant la mise à jour puis mes retours.

[blues]

J'avais des scripts en stock. Je ne vais plus oser les sortir ! [FACE WITH TEARS OF JOY]
Allez, les barbus, on se sert la main et on oublie cette première rencontre.
J'en profite pour remercier Influman pour ses nombreuses contributions. Je ne suis pas encore utilisateur mais je commence à regarder.

[influman]

Comme souvent sur du code open que je suis amené à toucher, j'ai fait un retour, ce qui me semble justement le principe de la coopération open source. N'ayant pas de donnée git dans le code, je suis venu sur le forum et j'ai trouvé ce sujet.

Ce n'est pas le principe d'aider et conseiller, ça c'est génial, c'était juste la forme qui était maladroite, mais aucune espèce d'importance sur le fond, passons au pragmatique.

J'ai vu ton pull request sur git, mais comme je te dis, je ne suis pas du tout utilisateur de ce système (je me suis limité au partage ), il faut que tu me dises quoi faire maintenant dans Git pour intégrer tes modifs proposées et les retrouver ensuite dans ma partie locale...

Par ailleurs, il faut aussi modifier le json associé, pour monter le numéro de version et corriger la description, pour qu'il soit intégrable dans le store. Mais ça je pourrais le faire.

Si t'es chaud, pourras-tu également regarder du côté des petits scripts suivants qui, même s'ils marchent, ont des pb de latence je trouve, alors que de la réactivité est attendue :
- ToggleChain (forcément )
- Pushbutton
- Notification IFTTT
- Double détection

et le gros script tordu artisanal :
- Chatbot

Les autres scripts ont sans doute des besoins d'optimisations similaires, mais l'éventuelle latence n'est pas spécialement problématique pour l'usage. Donc non prioritaire.

thx

[jbourdin]

Déjà, sur toggle, j'ai vu que la v1.1 était remontée, tu as du avoir une synchro qui est remontée entre ton repo local et celui chez github. J'ai mis à jour de mon côté pour que ça reste cohérent.
Donc au final, ça fait un pull request (en gros, une invitation à récupérer le code depuis une copie du repository) que tu peux regarde ici https://github.com/influman/toggle/pull/1/files et qui te permet de bien lire les changements.
Si cela te convient, il te suffit de cliquer sur "merge" et ça rapatrie les changements de ton côté. Sur le fond, pour toggle, je n'ai fait que ranger le code mais il n'y a aucune latence due au code justement, tout est dans la chaine amont avant le déclenchement du script chez moi. Donc inclure mes changements, ça n'apporte rien sur le fond.

Pour aller plus loin sur git, la doc référence (en français ou en anglais) : https://git-scm.com/book/fr/v2. Un peu long mais très complet.

Je vais essayer de jeter un œil sur les autres que tu cites mais ce qui me concerne le plus, ce sera plutôt le suivi de consommation et là, il y a plus de code. (et chatbot, je n'ai pas d'équipement pou tester)
J'envisage de faire un périphérique "estimation de consommation" pour les radiateurs électriques pilotés (fil pilote ou relai) puisqu'avec la puissance de chacun et les périodes on/off, on peut recalculer facilement une consommation. A voir donc.

[jbourdin]

J'ai fait une petite simplification de code sur Togglechain aussi
https://github.com/influman/togglechain/pull/1/files

N'hésite pas si tu as des questions