Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

problème de gestion des Utilisateurs entre user-seen-at-home et back-at-home #262

Closed
Jean-PhilippeD opened this issue Dec 17, 2017 · 3 comments

Comments

@Jean-PhilippeD
Copy link
Contributor

Jean-PhilippeD commented Dec 17, 2017

Salut,
En travaillant sur une fonction de purge des event, j'ai purgé trop d'events :)

Du coup, en relancant mon instance, n'ayant aucun d'évenement back-at-home ou left-home, je suis donc considéré à juste titre comme user-seen-at-home

Sauf quand mon controle de présence sur détection de mouvement appel la fonction house.isAtHome qui appelle elle meme la fonction house.getUsers, hors cette dernière s'appuie sur les évènements back-at-home et left-home et donc visiblement pas sur user-seen-at-home.

Ainsi, je suis considéré comme n'étant pas à la maison tant que je n'y serais vraiment pas afin de reprendre un état normal.

Si mon analyse est confirmée, ne faudrait-il pas intégrer l'event type user-seen-at-home dans la fonction house.getUsers ?

@Pierre-Gilles
Copy link
Contributor

Mmmm on pourrait, après c'est plutôt ta fonction de purge qui devrait être remise en cause :p

Un event "user-seen-at-home" déclenche nécessairement un event "back-at-home" si tu n'es pas à la maison normalement, donc ce cas n'arrive pas trop en général ^^

Mais bon ça pourrait être rajouté !

@Jean-PhilippeD
Copy link
Contributor Author

Ben en fait j'ai fais une purge manuelle trop agressive.
Coté fonction, c'est censé etre clean, je vais l'éprouver ces prochains jours.

Mais dans l'opération, j'ai perdu les dernies events left et back at home.

J'ai eu l'impression qu'en labsence de ces events, j'ai eu un user-seen-at-home et rien d'autre.
Donc si j'ai bien vu, ca n'arrive que dans le cas d'un premier démarrage tant qu'il n'y a pas eu de détection du premier left-home, donc cas certe assez rare.

@Pierre-Gilles
Copy link
Contributor

Effectivement c'est le cas!

Je vais ajouter l'event à la fonction pour éviter tout problème à l'avenir :)

R6n0 pushed a commit to R6n0/Gladys that referenced this issue Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants