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

feat(counter): display best month ever on counter page #426

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Delapouite
Copy link
Contributor

Bonjour.

Cette PR répond à la question du meilleur mois jamais enregistré pour un compteur donné (record absolu).

Exemple avec le compteur du Quai Pierre Scize:

image

@ThibautChd
Copy link
Collaborator

Salut Bruno,

Cette PR peut être intéressante, à condition que la qualité des datas qu'on remonte soit irréprochable, sans artefact qui ferait exploser les données d'un mois de manière anormale. Hors, dans l'exemple que tu donnes, je suis presque certain que la valeur d'octobre 2018 n'est pas correcte. On le voit clairement dans la série de donnée d'octobre :
image

En attendant qu'on trouve une manière de post-traiter à la main certaines données bizarres, je crains qu'on affiche souvent une information fausse.

@benoitdemaegdt
Copy link
Owner

Hello 👋

je crains qu'on affiche souvent une information fausse.

Souvent je ne sais pas trop, ça me semble quand même assez rare ce genre d'anomalie.
En tout cas, ça ne me semble pas plus faux que d'afficher une barre rose dans le graphe ^^(elle aussi peut théoriquement être fausse)

@ThibautChd
Copy link
Collaborator

Certes, mais cette barre rose saute bien moins aux yeux, car il faut sélectionner le mois avec l'anomalie pour tomber dessus.
Avec cette PR, on se retrouve avec la valeur constamment à l'écran.

Ceci dit, ça peut être intéressant de pousser cette PR pour vérifier si cette anomalie revient souvent, quitte à bricoler des exceptions pour retirer les quelques mois anormalement élevés ?

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

Successfully merging this pull request may close these issues.

3 participants