Liste deroulante d'une condition incomplete?

BOnjour

un truc bete

dans cette liste il manque le truc ne contient pas ce qui permettrait d’activer le chauffage en mode eco quand pas neccessaire par exemple

merci

Je reviens à la charge

Car je ne sais pas comment faire pour mon chauffage je vaudrais une température mini de 16 degrés mais avec le système de scène qui ne possède pas de sinon je ne vois pas comment faire

J’ai réussi à faire un truc propre pour que le. Chauffage s’active à partir du calendrier mais sans cette condition ce obligé à faire un calendrier remplie alors que c’est juste quand il y a rien qu’il faut activé un truc

J’espère avoir été clair (pas sur)

Bon dimanche

Salut !
Chez moi, le chauffage est géré par Gladys.
Pour la température minimale, c’est une simple scène qui s’active si la température est inférieure à 17°C sans condition (enfin si, j’ai un mode ‹ hors gel ›).
Pour la désactivation de la chaudière, je coupe s’il y a 18°C ou plus et qu’on n’est pas en mode confort.
Déclencheur : température >18°C
Récupérer dernier état du mode de chauffage
Continuer seulement si différent de mode confort
Éteindre le chauffage.

Tout le détail est ici, dans la partie ‹ Programmation › :
https://community.gladysassistant.com/t/tutoriel-maj-thermostat-fait-maison-et-integration-programmation-dans-gladys

1 « J'aime »

je regarde demain a tete reposé
merci

J’ai fait un développement dans ce sens

Il y a une image de test si tu veux vérifier le fonctionnement cicoub13/gladys:feat-calendar-does-not-contain

:warning: Si plusieurs évènements ont lieu au moment de la scène, il suffit qu’un évènement ne contienne pas le mot recherché pour que la scène continue.

Je n’ai pas ajouté cette option au déclencheur (cela a beaucoup moins de sens pour moi).

Alors je pourrais pas test car la dernière fois

J’ai du tout refaire quand j’ai test une image

Il faut que je m’installe un deuxième machine de test pour cela

Désolé

Merci pour la PR @cicoub13 :slight_smile:

Petite question, pourquoi avoir fait ce choix et pas l’inverse ?

Est-ce que justement on ne veut pas que tous les évènements courant ne contiennent pas le mot ?

ça me parait plus logique pour les usages possible de cette nouvelle option mais je peux me tromper :slight_smile:

Effectivement, ne met pas en danger ta prod pour ce genre de test :slight_smile:

Cela me paraît plus logique que cela soit tous les événements qui ne contiennent pas…

2 « J'aime »

Bon, j’ai retourné la question dans tous les sens et ça me paraît plus logique comme vous dites :smile:

Scénario imaginaire

Imaginons que je veuille ne pas chauffer chez moi quand c’est jour tempo Rouge, je vais configurer un scène qui chauffe tous les jours avec l’action suivante

image

Tout fonctionne niquel, en jour bleus/blancs, la scène continue car « un évènement qui ne contient pas rouge a été trouvé ».

Lors d’un jour rouge, la scène s’arrête car « aucun évènement qui ne contient pas rouge n’a été trouvé » :exploding_head:

Maintenant, si j’ajoute un rendez-vous le matin dans mon calendrier « RDV docteur », le comportement attendu est que la scène s’arrête toujours.

Mais du coup, je suis embêté avec la phrase de l’UX

Si un évènement est trouvé Continuer la scène

Cool ! Je peux review ta PR du coup ?

Pour ça je conseillerais de créer un calendrier spécifique, plutôt que de mélanger avec d’autres évènements qui pourrait intérférer.

PS: Je rappelle que pour Tempo EDF on a déjà des conditions de scène natives :smiley:

Non, je n’ai pas fait les changements encore. Ça casse un peu le fonctionnement juste pour cette condition. Je suis dessus

1 « J'aime »

C’est codé, testé et poussé mais c’est vraiment une façon de penser bizarre (à l’envers je dirais).
J’ai peur que quelques personnes utilisent cette feature mais soient surpris du résultat :thinking:

je pensais que tu étais convaincu ? :joy:

Je t’ai fais une review:

C’est corrigé, je te laisse revenir vers moi si besoin

Review faite!

Bon, j’ai revu ce matin mon code avec un cerveau frais et c’est plus clean :slight_smile: Preneur de tes retours

2 « J'aime »

Merci! Côté fonction findCurrentlyRunningEvent ça me parait beaucoup mieux :slight_smile:

J’ai 2 petits commentaires sinon

1 « J'aime »

Pris en compte

Merci pour les retours, je trouve ça cool ! Par contre c’est vraiment une fonctionnalité qui donne des noeuds au cerveau :sweat_smile:

Pour info, la version finale ressemble à ça :

Est-ce qu’il faudra pas un test réel de cette fonctionnalité avant de pousser en prod ? Où à minima quelqu’un qui nous donne son cas d’usage précis et on le teste ?

@Prof_Techno chez toi, c’est quoi le cas ? A quoi ressemble ton calendrier ?

Ça me va d’attendre un cas réel. J’avoue que je suis parti vite dans le code avant de bien comprendre le besoin (pas bien)

1 « J'aime »