¿Lista desplegable de una condición incompleta?

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

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…

Bueno, he dado vueltas a la pregunta en todos los sentidos y me parece más lógico como dices :smile:

Escenario imaginario

Imaginemos que no quiero calentar en mi casa cuando es día tempo Rojo, configuraré una escena que calienta todos los días con la siguiente acción

Todo funciona genial, en días azules/blancos, la escena continúa porque « se ha encontrado un evento que no contiene rojo ».

En un día rojo, la escena se detiene porque « no se ha encontrado ningún evento que no contenga rojo » :exploding_head:

Ahora, si añado una cita por la mañana en mi calendario « Cita con el doctor », el comportamiento esperado es que la escena se detenga.

Pero entonces, estoy en un aprieto con la frase de la UX

Si se encuentra un evento Continuar la escena

¡Genial! ¿Puedo revisar tu PR entonces?

Para eso, te aconsejaría crear un calendario específico, en lugar de mezclar con otros eventos que podrían interferir.

PD: Recuerdo que para Tempo EDF ya tenemos condiciones de escena nativas :smiley:

No, aún no he hecho los cambios. Rompe un poco el funcionamiento solo para esta condición. Estoy en ello.

Está codificado, probado y subido, pero es una forma de pensar realmente extraña (al revés, diría yo).
Temo que algunas personas usen esta función, pero se sorprendan del resultado :thinking:

¿Pensé que estabas convencido? :joy:

Te hice una revisión:

https://github.com/GladysAssistant/Gladys/pull/2187#pullrequestreview-2575972478

Ya está corregido, te dejo volver a mí si necesitas

¡Revisión hecha!

Bueno, revisé mi código esta mañana con la mente fresca y ahora está más limpio :slight_smile: Tomaré tus comentarios

¡Gracias! La función findCurrentlyRunningEvent me parece mucho mejor :slight_smile:

Tengo 2 comentarios más.

Tomado en cuenta

¡Gracias por los comentarios, me parece genial! Aunque es una función que da dolor de cabeza :sweat_smile:

Para información, la versión final se ve así:

¿No deberíamos hacer una prueba real de esta función antes de desplegarla en producción? O al menos alguien que nos dé un caso de uso preciso y lo probamos?

@Prof_Techno, en tu caso, ¿cuál es el caso de uso? ¿Cómo es tu calendario?

Me parece bien esperar a un caso real. Reconozco que me lancé rápido al código sin entender bien la necesidad (no fue lo ideal)