# Philips Hue Fade-in **URL:** \ **Category:** Développement\ **Created:** [Janvier 14, 2021, 7:35 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008 "2021-01-14T19:35:17Z")\ **Posts on this page:** 20\ **Page:** 1
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Janvier 14, 2021, 7:35 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/1 "2021-01-14T19:35:17Z")
Salut ! Ce sujet concerne la faculté de programmer un allumage progressif d’une ampoule Hue, comme présenté dans cette [issue](https://github.com/GladysAssistant/Gladys/issues/823) Je n’ai pas eu beaucoup de temps à passer sur ce sujet, mais j’ai une première ébauche qui est fonctionnelle. Je pense qu’elle nécessite discussion avant de poursuivre et de finir tout ce qui est nécessaire pour que la [PR](https://github.com/GladysAssistant/Gladys/pull/1041) soit valide. Les sujets qui portent à discussion : - Il n’y a pas encore dans le projet la notion de “scénario” associée à une Scène. Jusqu’à présent toute scène est composée d’une suite d’actions basiques comme allumer une lumière ou actionner un interrupteur. Faire un fade-in demande d’enchaîner plusieurs actions de bases, l’API Hue ne proposant pas d’action directe pour faire un allumage progressif. Ici on a donc besoin de chaîner plusieurs actions simples : allumer la lumière puis changer la luminosité à intervalles réguliers. Ce “scénario” demande un jeu de paramètres plus évolué que pour les actions basiques : on a besoin à la fois de la durée de l’allumage et de la luminosité cible. Ce n’est pas possible d’intégrer ces paramètres dans le modèle actuel et j’ai dû tricher en modifiant la validation du format des actions d’une scène pour avoir quelque chose qui tourne. Je pense donc qu’il faut réfléchir à quelque chose d’adapté. - Côté front, j’ai encore dû tricher pour le Component LightFadeInParams pour gérer cette “action” avec plusieurs paramètres. A discuter aussi pour faire quelque chose de plus propre. - J’ai joué avec des setTimeout pour faire l’allumage progressif mais je n’aime pas trop cette solution : on risque d’avoir un allumage qui finit en retard. Il faudrait sûrement s’aider d’une librairie plus adaptée. S’il n’y a pas de package adapté déjà utilisé dans Gladys, je regarderais ce qui pourrait faire l’affaire. De plus, quand le temps d’allumage est court, l’API Hue semble changer l’état de la lumière un peu en retard par rapport au retour d’appel. Ce qui provoque des “saccades” dans le processus d’allumage progressif. Je vais regarder pour peut-être limiter le nombre d’appels à l’API. Il y a pas mal d’autres choses à revoir sur la PR mais je préfère discuter de ces sujets avant d’aller plus loin. ---
**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\ **Post date:** [Janvier 15, 2021, 8:09 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/2 "2021-01-15T08:09:05Z")
Salut @christophe hésite pas à faire signe si tu as besoin de testeur ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Janvier 15, 2021, 8:38 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/3 "2021-01-15T08:38:54Z")
Salut @christophe et top si tu as pris le développement de cette PR 🙂 > [@christophe](#): > > l’API Hue ne proposant pas d’action directe pour faire un allumage progressif. Tu es sûr de ça? En cherchant rapidement, je suis tombé sur ça: > ![Screenshot 2021-01-15 at 09.35.26](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/d/d608e0ed5d6f95170a13fb4c9dd7aa8a96fe0740.png) ça ressemble à ce que l’on cherche non ? > [@christophe](#): > > Ce n’est pas possible d’intégrer ces paramètres dans le modèle actuel et j’ai dû tricher en modifiant la validation du format des actions d’une scène pour avoir quelque chose qui tourne. Je pense donc qu’il faut réfléchir à quelque chose d’adapté. ce n’est pas « tricher », c’est comme ça qu’on développe à chaque fois ! Si tu as besoin de plusieurs attributs dans une action, c’est normal de rajouter ces attributs à la validation. > [@christophe](#): > > Côté front, j’ai encore dû tricher pour le Component LightFadeInParams pour gérer cette « action » avec plusieurs paramètres. A discuter aussi pour faire quelque chose de plus propre. Tu pourrais montrer directement le code pour montrer en quoi c’est « tricher » ? 🙂 ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Janvier 15, 2021, 4:12 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/4 "2021-01-15T16:12:05Z")
Haha effectivement je ne me suis attardé que sur l’API sur le site Philips et n’ai pas été assez curieux pour regarder ce que proposait le module node directement. My bad, je vais regarder pour l’utiliser. Pour préciser mon propos sur où j’ai modifié la validation du modèle : [lien](https://github.com/GladysAssistant/Gladys/pull/1041/files#diff-fe714b595358f083938aa1894f261939e53ba07352550bbdcdef293add71dc8cR18) Voilà où j’ai triché dans le front, dans l’implémentation du formulaire de création d’action de fade in : [lien](https://github.com/GladysAssistant/Gladys/pull/1041/files#diff-44bb39134ef751545fdc523ba94599e382b9c0baa749470c76c79179e695a2a7R8-R23) Cela est lié à la validation du modèle d’une scène, comme j’ai ajouté un attribut “parameters” pour gérer l’ensemble de mes valeurs (voir le lien précédent), je mets à jour l’attribut “parameters” en entier à chaque fois plutôt que de ne mettre à jour qu’un sous-attribut de “parameters”. C’est pas hyper clean, mais en l’état actuel je ne vois pas comment faire autrement. ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Janvier 15, 2021, 4:12 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/5 "2021-01-15T16:12:35Z")
@VonOx merci, je te solliciterai quand ce sera un peu plus stable 🙂 ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Janvier 15, 2021, 4:57 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/6 "2021-01-15T16:57:46Z")
> [@christophe](#): > > Voilà où j’ai triché dans le front, dans l’implémentation du formulaire de création d’action de fade in : [lien](https://github.com/GladysAssistant/Gladys/pull/1041/files#diff-44bb39134ef751545fdc523ba94599e382b9c0baa749470c76c79179e695a2a7R8-R23) Ah effectivement c’est pas fou, pourquoi tu n’as pas fais 2 variables ? pas besoin d’une variable parente « parameters ». On le fait dans plusieurs actions de scène pourtant. Exemple dans ton cas: ```auto handleChangeTargetBrightness = e => { this.props.updateActionProperty(this.props.columnIndex, this.props.index, 'target_brightness', e.target.value); }; ``` D’ailleurs, je remarque que tu fais un setState en plus de ton updateActionProperty, c’est pas terrible car ça t’oblige à synchroniser les 2 états. Tu pourrais directement utiliser la props `props.action` qui est injecté par le parent ! ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Janvier 16, 2021, 7:52 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/7 "2021-01-16T19:52:40Z")
Si j’utilise trois variables, je dois modifier l’`actionSchema` du modèle `scene` et ajouter trois attributs `target_brightness`, `duration_value` et `duration_unit`. Ces attributs ne seront utilisés que pour ce type d’action. D’où l’idée d’utiliser un unique attribut `parameters` de type `object` pour rendre le schéma générique et réutilisable bien que très permissif. ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Janvier 18, 2021, 9:06 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/8 "2021-01-18T09:06:38Z")
> [@christophe](#): > > Ces attributs ne seront utilisés que pour ce type d’action. c’est justement le but ! 😃 > [@christophe](#): > > D’où l’idée d’utiliser un unique attribut `parameters` de type `object` pour rendre le schéma générique et réutilisable C’est ce qu’on veut éviter ! Dans Gladys 4, on fait tout au plus stricte. Si le schéma est trop souple, on va se retrouver avec n’importe quoi en DB, on va avoir des utilisateurs avec des problèmes mystiques, et ça sera très dur à débugger et à corriger. Là ou je suis d’accord avec toi, c’est que ce schema qui valide de la même manière toutes les actions c’est bizarre et pas assez stricte. Je pensais pas que c’était possible avec Joi au début, mais en fait on peut faire des schémas conditionnels avec .when() ( [joi.dev - 17.12.0 API Reference](https://joi.dev/api/?v=17.3.0#anywhencondition-options) ) J’aimerais bien migrer vers ça pour qu’on puisse valider individuellement chaque type d’action. ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Janvier 22, 2021, 6:46 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/9 "2021-01-22T18:46:18Z")
Voilà pourquoi je voulais en discuter, je n’aime pas l’idée d’avoir des options qui seront null la plupart du temps et je pense aussi que parameters est trop permissif. Je vais regarder les schémas conditionnels. ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Janvier 24, 2021, 6:06 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/10 "2021-01-24T18:06:07Z")
Salut, J’ai mis à jour la PR avec : - L’utilisation directe de la fonction de l’API Hue pour préciser la durée du fade in - L’utilisation des schémas conditionnels pour les paramètres des scènes. Pour cette dernière, j’ai gardé le champ parameters pour englober les anciens champs text, value et unit. J’ai créé une migration en conséquence. Il me reste encore un peu de travail pour rendre tout ça plus complet et testé. Sans parler des pré-requis pour que la PR soit valide 🙂 ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Janvier 24, 2021, 6:11 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/11 "2021-01-24T18:11:10Z")
Génial ! Hâte de voir ça 🙂 ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Mai 13, 2021, 9:25 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/12 "2021-05-13T09:25:56Z")
Salut, Après une hibernation de 4 mois j’ai repassé un peu de temps sur la PR. Je pense qu’elle est prête pour une review, j’ai construit une image pour mon Rasp 4 et ça tourne bien. @VonOx si tu souhaites tester. Merci @pierre-gilles et @AlexTrovato pour la première relecture, j’ai pris en compte les retours. ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mai 14, 2021, 3:05 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/13 "2021-05-14T03:05:03Z")
Salut Christophe! Déjà merci pour ta PR et tout le travail réalisé dessus, c’est cool 🙂 J’ai écris une review sur ta PR. > > > \### Pull Request check-list > > To ensure your Pull Request can be accepted as fa…st as possible, make sure to review and check all of these items: > > \- \[x\] If your changes affects code, did your write the tests? > \- \[x\] Are tests passing? (\`npm test\` on both front/server) > \- \[x\] Is the linter passing? (\`npm run eslint\` on both front/server) > \- \[x\] Did you run prettier? (\`npm run prettier\` on both front/server) > \- \[x\] If you are adding a new features/services, did you run integration comparator? (\`npm run compare-translations\` on front) > \- \[x\] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code) > \- \[x\] If you are adding a new features/services which needs explanation, did you modify the user documentation? See \[the GitHub repo\](https://github.com/GladysAssistant/v4-website) and the \[website\](https://gladysassistant.com). > \- \[\] Did you add fake requests data for the demo mode (\`front/src/config/demo.json\`) so that the demo website is working without a backend? (if needed) See \[https://demo.gladysassistant.com\](https://demo.gladysassistant.com). > > NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open. > > \### Description of change > > Implements the possibility to create a scene with an action which turns on a Hue light progressively. Ta PR fait des changements assez majeure sur l’API à pas mal d’endroits qui ne sont pas lié au Philips Hue et je pense que ça vaudrait le coup d’en discuter avant, et si ces changements sont vraiment nécessaires (je ne suis pas sûr), je pense que ça a plutôt sa place dans une PR séparé. On essaie de faire des PR plutôt petites en général, et quand il y a des changements d’API majeure (comme ici dans ta PR) avec des migrations, c’est quelque chose dont on discute en général avant. Je pense que c’est super important d’avoir une API stable et non breaking dans le temps, car des utilisateurs en dépendent, et codent potentiellement des scripts, des intégrations externe qui en dépendent, donc ce n’est pas à prendre à la légère ce genre de changements. ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Mai 14, 2021, 8:20 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/14 "2021-05-14T08:20:57Z")
Salut, J’avais abordé plus haut dans cette conversation pourquoi j’avais besoin de faire ces modifications. J’ai besoin de trois paramètres pour configurer mon action : l’intensité cible, l’unité de temps et la valeur pour la durée de l’action. L’objet action n’était pas adapté selon moi : - Les seules valeurs possibles étaient text, value et unit ne donnant aucune information sur la nature réelle du contenu de la variable. - Ces valeurs se trouvent au même niveau que type et devices, on n’a pas de distinction entre les paramètres qui définissent la nature d’une action et ceux qui permettent à une action d’un certain type de tourner. En revanche je n’avais pas pris en compte le fait que l’API peut être consommée par d’autres consommateurs que le front de Gladys. Je pense qu’il faut effectivement que je revienne sur les modifications de l’interface de l’API, mais je propose de conserver la migration du modèle de données pour les raisons juste au dessus. L’introduction d’une notion de scenario permet de définir des actions plus complexes que simplement activer une feature. Je ne l’ai pas appelé fadeIn parce que la notion de scenario pourrait être utilisée pour n’importe quel type de device, le fadeIn par contre est spécifique aux ampoules. Après, je l’ai appelé scenario mais ça m’est égal d’utiliser un autre wording 😃 ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mai 14, 2021, 8:25 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/15 "2021-05-14T08:25:48Z")
L’objet action est tout fait extensible à souhait: tu mets ce que tu veux dedans 🙂 Si tu veux rajouter 3 attributs libre à toi! On avait discuté avant de rajouter des check spécifique à chaque action (au lieu de faire un check global), pas de créer un objet nested dans l’objet action. Pour moi, on gagnerait en lisibilité à garder un objet action « flat », pas besoin de créer des niveau supplémentaires. > [@christophe](#): > > L’introduction d’une notion de scenario permet de définir des actions plus complexes que simplement activer une feature. On a déjà une fonctionnalité de scénario dans Gladys donc ça porte à confusion vu que ça n’a pas de rapport ici. Si tu veux faire de l’ultra générique, pourquoi ne pas passer par le setValue dans ce cas là? ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Mai 14, 2021, 8:56 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/16 "2021-05-14T08:56:01Z")
Une des difficultés que j’ai rencontrée pour rentrer dans le concept de scènes et actions est justement ce manque de hiérarchie dans les objets action. Je trouve qu’un objet flat rend compliqué la compréhension sur le rôle de chaque propriété. Ma compréhension du setValue est le passage d’une feature d’un état à un autre pour un device donné. L’appel déclenche le changement d’état et retourne directement. Cela permet de répondre à un appel d’API dans un temps raisonnable. Ici, l’action fait partie d’une scène qui peut être composée de plusieurs groupes d’actions qui doivent s’exécuter séquentiellement. Imaginons une scène où je veux que mon ampoule commence à s’allumer à 7h20 pendant 10 minutes et qu’ensuite ma radio s’allume. Avec un setValue qui retourne directement, ma radio s’allumerait à 7h20. J’ai donc besoin d’attendre que l’état de mon ampoule soit l’état final désiré pour mettre fin à mon action et déclencher la suivante. De plus, dans mon “scenario” je fais appel à deux features différentes de l’objet light : binary et brightness. Le setValue devrait rester sur une seule feature. ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mai 14, 2021, 10:41 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/17 "2021-05-14T10:41:50Z")
> [@christophe](#): > > Une des difficultés que j’ai rencontrée pour rentrer dans le concept de scènes et actions est justement ce manque de hiérarchie dans les objets action. Je trouve qu’un objet flat rend compliqué la compréhension sur le rôle de chaque propriété. Je suis pas d’accord, chaque action est pour moi un objet, qui a toute ses propriétés. Un ensemble d’exemples: **Action « attendre 10 seconds »:** ```auto type: "wait" unit: "seconds" value: 12 ``` **Action lancer la scène « chauffage »:** ```auto type: "scene.start" scene: "chauffage" ``` **Condition « vérifier que l’heure est entre 12:00 et 14:00 »:** ```auto type: "condition.check-time" after: "12:00" before: "14:00" ``` Je pense que si jamais à l’avenir quelqu’un code un petit tool pour faire des scènes en YAML et les importer dans Gladys, ça sera plus lisible. Après bon c’est un peu les goûts et les couleurs, je pense c’est aussi bien de garder quelque chose de léger, sans trop d’objets à instancier, et pas trop nested ^^ > [@christophe](#): > > Ma compréhension du setValue est le passage d’une feature d’un état à un autre pour un device donné. L’appel déclenche le changement d’état et retourne directement. Cela permet de répondre à un appel d’API dans un temps raisonnable. Dans l’API oui, après dans une scène tu peux faire un setValue qui attend que le device ait répondu, tout est possible 🙂 > [@christophe](#): > > De plus, dans mon “scenario” je fais appel à deux features différentes de l’objet light : binary et brightness. Je pensais que tu passais par l’API de Philips Hue qui fait ça en une commande ? ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mai 14, 2021, 10:45 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/18 "2021-05-14T10:45:36Z")
Ne prend pas mal mes remarques hein, je trouve ça super d’être challengé sur les choix fait à différents endroits 😃 Mais je défend aussi les choix de format de donnée choisi, ça n’a pas été choisi à la légère et je pense que c’est important de rester rétro-compatible dans la mesure du possible (à part si vraiment c’est méga bloquant et qu’on veut avancer). Je pense que c’est un bon signal à envoyer à la communauté que l’API est “stable” et ne change pas dans des versions mineurs de Gladys tous les 4 matins 🙂 ---
**Author:** ![christophe](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/christophe/32/5992_2.png) [@christophe](https://community.gladysassistant.com/u/christophe)\ **Post date:** [Mai 14, 2021, 1:04 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/19 "2021-05-14T13:04:50Z")
Ok je vais retirer le parameters alors. J’ai besoin d’allumer la lampe et de réinitialiser l’intensité au minimum au début de l’action avant d’utiliser la commande de changement d’intensité progressif. Je change donc plusieurs états sur des features différentes d’un même device. ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [Mai 15, 2021, 1:57 UTC](https://community.gladysassistant.com/t/philips-hue-fade-in/6008/20 "2021-05-15T01:57:29Z")
Ok je vois Après ça c’est un « détail technique » de l’implémentation Philips Hue non? Ce n’est pas propre à Gladys ? Bon je verrais dans ta PR ce que tu as fais 🙂 > [@christophe](#): > > Ok je vais retirer le parameters alors. Top! Merci pour le changement 🙏