-
Notifications
You must be signed in to change notification settings - Fork 3
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
When generating yearly reports, warn and abort if there are less than 100 valid data days #292
Conversation
@spch-GL Navré mais le commentaire très utile que tu as fait sur cette même Pull Request semble ne plus être visible (si c'est parce que j'ai basculé la Pull Request en mode "ébauche", je m'en excuse). Oserais-je te demander de bien vouloir me dire, toujours sur cette même PR plutôt qu'en réponse à ce courriel, quelle logique tu souhaiterais idéalement? Une erreur? Un avertissement avec une tentative de quand même générer le rapport? Un effort du système pour générer le rapport quoi qu'il en soit sauf problème majeur? Bien cordialement |
@why-not-try-calmer Je ne crois pas avoir fait de commentaire sur un PR mais seulement sur des issues... Dans l'idéal, il faudrait avertir l'utilisateur qu'il n'y a pas assez de données en indiquant le nombre de jours pris en compte, et lui demander si il veut tout de même générer le rapport annuel oui ou non ("non" par défaut), le tout dans une seule boite de dialogue. |
Très bien, et "pas assez de données" serait défini comme "moins que le maximum possible" (incomplet) ou autrement? |
extrait de #273 : ... pas de rapport annuel avec C'est la définition de "pas assez de données" Mais il reste à déterminer comment vérifier si les données fournies sont assez complètes ... |
A très bien, c'est précisément cette définition dont je craignais qu'elle ait été perdue avec un commentaire. Tout est clair, je ferai une proposition pour le seuil "assez complètes" |
e669965
to
20f7ea2
Compare
eb9ee79
to
44f9ee5
Compare
Salut Adrien,
Désolé, j’étais en séance et ne vois ta question que maintenant…
Je n’ai pas retrouvé ton commentaire dans la PR 292… alors je te réponds par courriel
Je ne vois pas ton intervalle 14-38…
Mais ce que je comprends :
Hour : intervalle horaire entre x et x+1
THM : nombre de véhicules pendant cette heure (Trafic Horaire Moyen)
Par contre date, c’est plus étrange…
Nous avons les jours de la semaine (0-6 ou 1-7)
Mais cela pourrait aussi être les jours ouvrables/les week-end/les jours fériés
Pierre Grellet
Chef de projets en informatique technique (DDTE - SPCH - OSMO)
De : Adrien ***@***.***>
Envoyé : jeudi, 23 novembre 2023 11:08
À : opengisch/OpenComptage ***@***.***>
Cc : Grellet Pierre ***@***.***>; Mention ***@***.***>
Objet : Re: [opengisch/OpenComptage] When generating yearly reports, warn and abort if there are less than 100 valid data days (PR #292)
[E-MAIL EXTERNE]
Je ne comprends pas ce dataframe Panda:
date hour thm
0 1 0 7
1 1 1 6
2 1 2 12
3 1 3 40
4 1 4 229
5 1 5 755
6 1 6 744
7 1 7 488
8 1 8 423
9 1 9 419
10 1 10 535
11 1 11 429
12 1 12 462
13 1 13 313
14 4 14 121
15 4 15 547
16 4 16 631
17 4 17 520
18 4 18 393
19 4 19 200
20 4 20 88
21 4 21 70
22 4 22 58
23 4 23 35
24 6 0 102
25 6 1 64
26 6 2 52
27 6 3 19
28 6 4 20
29 6 5 19
30 6 6 59
31 6 7 84
32 6 8 165
33 6 9 262
34 6 10 391
35 6 11 535
36 6 12 365
37 6 13 323
38 6 14 352
obtenu par la fonction get_time_data_yearly(2021, section) où section est le fameux tronçon 0010769. Que représente l'intervalle "14-38" (colonne "heure")?
—
Reply to this email directly, view it on GitHub<#292 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AGJHY27IVIK762WCRM6OYLLYF4N6XAVCNFSM6AAAAAA7K766X6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRUGEYTGOJQHA>.
You are receiving this because you were mentioned.Message ID: ***@***.******@***.***>>
|
Pas de problème, commentaire effacé, cette question-là est réglée |
6ee6dfa
to
c2bf1d8
Compare
…back on exceptions
8cffcec
to
4e66666
Compare
Désolé, je ne suis de retour que maintenant et ne trouve pas ce 25... |
This comment was marked as resolved.
This comment was marked as resolved.
Okay on est bon, c'est déjà disponible dans la release à laquelle je t'ai référé dans mon email de tout à l'heure. |
Avec la version 2.1.3b, le rapport annuel n'est pas généré, il y a une erreur de
Le code passe donc toujours par le groupBy or tu as ajouté de nouvelles fonctions et il me semble qu'elles ne sont pas utilisées |
J'ai oublié de préciser que le cas ci-dessus, c'est pour le cas spécial EGAS et qu'il n'a pas proposé le choix des rapports à produire
|
Il y a bien du code inutilisé mais pas de groupby de ma part. L'erreur ci-contre ne fait pas partie du code que j'ai modifié. |
Justement, je pensais que tu avais créé les fonctions ( |
Non j'ai écrit |
Après avoir ajouté un print()
Seuls les rapports des 2 sections 00056202 et 00056200 ont été générés normalement. Remarque : |
Encore une précision: j'ai encore testé sur la section 00107695, et le print() donne: Ce que je remarque, c'est que lorsque qu'il n'y a pas de rapport produit, le nombre de voies de la section est impaire ! |
Merci pour tes tests, c'est intriguant en effet. Je vais regarder. |
|
Oui et non ! |
Est-ce que les rapports EGAS sont générés correctement sur la dernière version officielle (sans mes changements)? |
Non, c'est pourquoi il y a des |
Okay, alors si ce problème n'est pas lié spécifiquement à cette PR, est-ce qu'on peut valider cette PR? Cela impliquerait que je mets le label "reviewed" et que je passe à la suite, pour ensuite refaire une release p1, puis enfin réitérer le processur avec p2. |
Presque...
|
2d61de0
to
674f440
Compare
No description provided.