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

Suppression du cache de l'indice de satisfaction et nombre d'avis d'une démarche calculé à base des données des avis détaillés #1182

Open
lucaa opened this issue Aug 7, 2022 · 5 comments
Assignees
Labels
agrégats avis Tâches liées au calcul et utilisation des données agrégées des avis. mode connecté Amélioration ou problème qui impactent les fonctionnalités pour les utilisateurs connectés. poker 2 scalabilité/performance Tickets liés à la performance de la plateforme, liée à l'accumulation des avis.
Milestone

Comments

@lucaa
Copy link
Collaborator

lucaa commented Aug 7, 2022

Un cache des donnés des avis d'une démarche a été ajouté au début du projet ( #242 ), utilisé pour la première version du panneau de suivi des démarches, aujourd'hui accessible uniquement aux adminstrateurs.

Ce cache stocke: le nombre d'avis d'une démarche, l'indice de satisfaction et le nombre de réponses 'oui' à la question 'aurez-vous préféré faire la démarche en ligne', non-utilisé aujourd'hui - voir aussi #1114 .
Le cache est calculé tous les jours (?).

Cette tâche est pour enlever ce calcul qui n'est plus nécessaire suite à la mise en place de #1129:

  • soit remplacer ce calcul avec un calcul à base des agrégats
  • soit supprimer complètement ce cache qui n'est plus nécessaire - et supprimer aussi ses usages ou bien remplacer ses usages avec les données des agrégats
@lucaa lucaa added the mode connecté Amélioration ou problème qui impactent les fonctionnalités pour les utilisateurs connectés. label Aug 7, 2022
@lucaa lucaa added this to the 5.0 milestone Aug 7, 2022
@lucaa lucaa self-assigned this Aug 7, 2022
@lucaa lucaa added the scalabilité/performance Tickets liés à la performance de la plateforme, liée à l'accumulation des avis. label Aug 7, 2022
@lucaa
Copy link
Collaborator Author

lucaa commented Aug 7, 2022

@AnthonyBrunelli @jrenauxdinum aujourd'hui le seul usage de ce cache, l'ancien 'tableau de bord' - la première version, qui affiche les démarches en direct - n'est disponible qu'aux administrateurs ministériels et administrateurs dinum, à partir de la page d'accueil.

A-t-on des informations liée à l'utilisation de cette fonctionnalité pour savoir si on peut enlever complètement cette fonctionnalité?
Aussi, je ne suis pas sure que les données de ce cache sont 100% corrects, car son calcul n'a pas été surveillé depuis le remplacement de ce tableau de bord par l'observatoire.

Il faudra aussi consulter les statistiques d'accès pour voir si des visualisations sont enregistrées pour cette page.

@AnthonyBrunelli
Copy link
Collaborator

A priori, tout indique qu'on peut supprimer. Nous n'avons pas trouvé avec Joffrey où était le tableau concerné. On confirme lors de notre point de vendredi.

@lucaa
Copy link
Collaborator Author

lucaa commented Aug 31, 2022

@AnthonyBrunelli il y a un lien vers ce tableau sur la page d'accueil connecté, pour l'admin dinum (et les admins ministeriels, si je me rapelle correctement):

image

Ensuite, dans le tableau affiché à partir de ce lien, on voit un indice de satisfaction qui est basé sur ce cache dont parle ce ticket:

image

Comme je le disais dans mon commentaire ci-dessus, je ne suis pas sure que ces valeurs sont 100% correctes, ou si elles concernent l'année glissante ou les données depuis le début (je dois vérifier l'implémentation et les tickets liés).

@AnthonyBrunelli
Copy link
Collaborator

On peut supprimer les colonnes dont on n'est pas sûr et les remplacer par les agrégats. J'ai vu aussi que l'indicateur "Top 250" pose question (il faudrait qu'il soit à "non" par défaut, sinon ça n'est pas renseigné).
Ce tableau est utile pour voir ce qui se fait hors top 250 sinon.
On en reparle vendredi.

@AnthonyBrunelli
Copy link
Collaborator

Vu ensemble aujourd'hui : on supprime ce tableau, on optimisera les tableaux issus des agrégats pour avoir les infos dont on a besoin - surtout si les informations ne sont pas fiables.

@lucaa lucaa modified the milestones: 5.0, 5.1 Nov 14, 2022
@lucaa lucaa added the agrégats avis Tâches liées au calcul et utilisation des données agrégées des avis. label Dec 1, 2022
@lucaa lucaa modified the milestones: 5.1, 5.2 Dec 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agrégats avis Tâches liées au calcul et utilisation des données agrégées des avis. mode connecté Amélioration ou problème qui impactent les fonctionnalités pour les utilisateurs connectés. poker 2 scalabilité/performance Tickets liés à la performance de la plateforme, liée à l'accumulation des avis.
Projects
None yet
Development

No branches or pull requests

2 participants