search

incubateur-ademe / nosgestesclimat-site

Estimez votre empreinte climat - le code du site
https://nosgestesclimat.fr
MIT License
70 stars 40 forks source link

Mes données locales semblent obsolètes par rapport aux dernières mises à jour #842

Open mlbiche opened 1 year ago

mlbiche commented 1 year ago

⚠️ Avant de créer une issue, lisez attentivement notre FAQ qui explicite un bon nombre des limites du test nosgestesclimat.fr.

Poser une question

En accédant de nouveau à nosgestesclimat.fr/profil après avoir mis à jour mon questionnaire aujourd'hui, je ne peux plus ouvrir la catégorie Alimentation à cause de règles inconnues.

En détail, voilà toutes les règles qui sont détectées comme inconnues dans mes données locales :

La plupart correspondent à des questions qui m'ont été posées aujourd'hui (reposées j'ai l'impression ?). Seule la question sur le petit-déjeuner ne m'a pas été reposée.

Les options sur l'électroménager et le numérique ont été cochées ou non (je peux vous donner le détail si ça vous aide).

Proposer une amélioration

Il semblerait qu'il y ait un problème de rétro-compatibilité avec ma donnée locale. J'aurais bien filé un coup de main là-dessus mais je ne sais pas quelle politique vous souhaitez appliquer vis-à-vis de ce problème.

Clemog commented 1 year ago

Salut @mlbiche !

Merci pour ton message et les PR c'est trop cool de voir ce genre de contributions :)

Effectivement, comme tu le comprends, le modèle de calcul évolue, les règles (clés) aussi ! Nous n'avons pas aujourd'hui de script de migration entre deux versions du modèle, c'est un pb que nous contournons aujourd'hui en essayant d'ignorer les règles qui ne sont plus dans le modèle, notamment avec getSafeRule mais semble-t-il insuffisant ;)

Effectivement, nous assumons que les questions sont reposées si un nom de règle change.

On regarde tes PR la semaine prochaine ;)

Clemog commented 1 year ago

https://github.com/datagir/nosgestesclimat-site/pull/836 a permis de résoudre un partie du problème (l'app ne crash plus normalement !), merci @mlbiche !

En revanche, on a un problème sur le comportement souhaité.

L'objectif est de reposer la question pour des règles qui changent ou des réponses devenues obsolètes. Ce n'est pas le cas : en effet si on prend l'exemple des petits déjeuners, dans le cas ou une situation présente une réponse du type alimentation . petit déjeuner . type . français alors que ce petit déjeuner n'est pas dans les rules, la règle alimentation . petit déjeuner . type reste dans les foldedSteps et la question n'est donc pas reposée ... et une ligne avec réponse vide apparait dans le profil. A investiguer !

mlbiche commented 1 year ago

Effectivement, parmi toutes les règles que j'ai mentionnées dans le ticket, c'est la seule qui ne m'ait pas été reposée ce jour-là. Et donc la seule qui entrainait un crash de l'application depuis la page Profil.

Clemog commented 1 year ago

@florianpanchout c'est le pb dont je te parlais qui est lié aux tests que tu as pu faire sur les personas :)

laem commented 1 year ago

Un exemple d'erreur silencieuse dans la console. On voit qu'avec une simulation de septembre 2022, facilement récupérable dans cette archive https://github.com/datagir/nosgestesclimat-site/issues/963, on obtient pas mal d'erreurs dans la console.

Une petite PR de nettoyage serait un luxe en effet.

image