22 onglet data ameliorations #29

tgazagnes · 2024-06-20T15:33:32Z

Améliorations validées avec Quentin et Téo, + conflits poetry.lock résolus

…iltres

…lectes

…rques)

tgazagnes · 2024-06-20T15:43:45Z

Cette fois le precommit check a marché 👯

KyllianBeguin · 2024-06-20T15:44:37Z

Sais-tu pourquoi ? 😅

tgazagnes · 2024-06-20T15:59:16Z

Des erreurs dans le fichier poetry.lock qui avait été forké depuis staging, et je n'avais pas pensé à faire un poetry install ou add pour vérifier que tout va bien avant

KyllianBeguin · 2024-06-21T21:20:22Z

dashboards/app/pages/data.py


                folium.CircleMarker(
                    location=(row["LIEU_COORD_GPS_Y"], row["LIEU_COORD_GPS_X"]),
                    radius=radius,  # Utilisation du rayon ajusté
-                    popup=f"{row['NOM_ZONE']}, {row['LIEU_VILLE']}, {row['DATE']} : {row['nb_dechet']} {selected_dechet}",
+                    popup=folium.Popup(
+                        html=f"""


Est-ce qu'on ne mettrait pas les html dans des fichiers dédiés ?

Ça concerne un seul graphique pour tout l'onglet, tu penses que c'est nécessaire ?

KyllianBeguin · 2024-06-21T21:30:31Z

J'ai fais le tour des fichiers. Dans l'ensemble, c'est plus des TO DO que des NO GO.
Énorme travail de refonte, merci Thibaut !

KyllianBeguin

✅

KyllianBeguin · 2024-06-21T21:21:22Z

dashboards/app/pages/data.py

+            if lieu != valeur_par_defaut_lieu:
+                filtered_df = filtered_df[filtered_df["TYPE_LIEU"] == lieu]
+
+        # The filtered_df now contains the data based on the selected filters


Pourquoi est-ce commenté ?
Si pas de raison spécifique => à supprimer

C'était du code que j'ai optimisé mais oublié de supprimer. C'est supprimé maintenant, merci !

KyllianBeguin · 2024-06-21T21:22:14Z

dashboards/app/pages/data.py

-            frenchify(nb_secteurs) + " secteurs",
-        )
+            # Ligne 2 : 3 cellules avec les indicateurs clés en bas de page
+            colors_map_secteur = {


À exporter dans un fichier de configuration style json?

Fait pour les 2 colormaps utilisées dans l'onglet, merci

KyllianBeguin · 2024-06-21T21:23:25Z

dashboards/app/pages/data.py

-        cell3.metric("Nombre de collectes comptabilisées", frenchify(nb_collectes_int))
+        cell3.metric("Nombre de ramassages", nb_collectes_int)
+
+        # Note méthodo pour expliquer les données retenues pour l'analyse


"Note méthodo" c'est pour faire une documentation pour expliquer la méthodo ?
Ou est-ce que ça a déjà été fait ?

Le commentaire était au mauvais endroit, merci pour la review. L'encart méthodo dans l'onglet est codé qq lignes plus bas

KyllianBeguin · 2024-06-21T21:23:50Z

dashboards/app/pages/data.py

-            options=["Aucune sélection"]
-            + list(filtered_data_milieu["TYPE_MILIEU"].unique()),
-        )
+            # Filter by year


Faudra voir si l'on garde les commentaires en français ou en anglais. Ça fait bizarre de passer de l'un à l'autre.

Yes you're right. J'ai tout passé en français

KyllianBeguin · 2024-06-21T21:25:05Z

dashboards/app/pages/hotspots.py

Pourquoi l'onglet hotspot est modifié, si l'on modifie data ?

C'est precommit qui a fait passer du reformatage (black je pense vu les modifs), je pense qu'ils ne l'ont pas utilisé dans les derniers commit du staging. Je les laisserai écraser les modifs avec leur dernière version pré-commitée :). Aucune modif volontaire de ma part

KyllianBeguin · 2024-06-21T21:25:21Z

dashboards/app/pages/hotspots.py

@@ -432,25 +432,30 @@ def calculate_and_display_metrics(data, indicator_col1, indicator_col2, indicato
 def couleur_milieu(type):


Prévoir du type hinting

à voir avec l'onglet hotspots

KyllianBeguin · 2024-06-21T21:26:14Z

dashboards/app/requirements.txt

@@ -7,7 +7,6 @@ streamlit==1.32.2
 openpyxl==3.1.2
 streamlit-folium==0.19.1
 plotly==5.19.0
-streamlit-dynamic-filters==0.1.6


Pas d'impact sur les autres onglets?

nope on l'avait testé sur data et remplacé ensuite. aucun autre onglet ne l'utilise

poetry.lock

KyllianBeguin · 2024-06-21T21:28:43Z

dashboards/app/pages/data.py

 import plotly.express as px
 import folium
 from folium import IFrame
+import math
+import locale
+import duckdb


Cool de voir du duckdb ! Peut-être à généraliser aux autres onglets?

Ok dans une V2 si ça te va, là je pense que l'enjeu est de boucler le POC

KyllianBeguin

OK

KyllianBeguin · 2024-06-29T15:50:27Z

dashboards/app/pages/ongletdata_colormap_materiaux.json

Note pour plus tard : Si l'on a d'autres export de colormap ou autre, autant tout centraliser dans un json ?

tgazagnes added 26 commits May 22, 2024 12:22

[tg] - valeurs par défaut des filtres et optimisation du layout des f…

fb186a5

…iltres

[tg] - convert liters to m3

2ad8ba5

[tg] - wording sous onglet matériaux

47049af

[tg] - classement par ordre alphabétique dans les filtres

1c9d037

[tg] - wording sous-onglet top déchets

b941905

[tg] - top déchets : passage rayon en log2

c19f74b

[tg] - chart labels templates

55e4894

[tg] - amélioration format nombres utilisant locale.set_locale

5c17732

[tg] - bug corrigé

042e133

[tg] - note méthodo et exclusion des 100%Autres

f224645

[tg} - infobulles dans les graphiques en langage naturel

e1c7a80

[tg] - onglet matériaux : tableau avec nb collectes par milieu

dc7a302

[tg] - onglet matériaux - améliorations table nb déchets

34b0996

[tg] - séparateurs décimaux et milliers dans les graphiques

9f1b0e4

[tg] - sous-onglet matériaux : filtrer sur milieux avec plus de 3 col…

0835d03

…lectes

[tg] - ajout des filtres sur top10 déchets

9225aea

[tg] inversion graphs marques et REP

3f24729

[tg] - sous-onglet 3 : filtre sur niveaux 4 (secteurs/REP) et 2à4 (ma…

5334ee7

…rques)

[tg] - retrait des VIDES et INDETERMINES dans les metrics de l'onglet 3

4c048eb

[tg] - ajout %déchets dans graph par secteurs

f7dc00c

[tg] - amélioration layout 3 onglets

fb43f53

[tg] - optimisation du code des filtres

a9ff5f1

[tg] Améliorations suite point Merterre

858bb7a

[tg] Ajout détail données utilisées + rayon carto top déchets

0c42b3b

[tg] remove line from poetry.lock to solve precommit fail

9ddfbed

[tg] update poetry.lock to solve precommit conflicts

ba83666

tgazagnes requested a review from KyllianBeguin June 20, 2024 15:33

tgazagnes self-assigned this Jun 20, 2024

[tg] black reformat to hotspots.py

8cc6880

tgazagnes marked this pull request as ready for review June 20, 2024 15:43

KyllianBeguin reviewed Jun 21, 2024

View reviewed changes

KyllianBeguin mentioned this pull request Jun 21, 2024

Amélioration de l'onglet Actions suite aux retours de MerTerre : #30

Merged

[tg] corrections suite retours Kyllian

e919f14

KyllianBeguin approved these changes Jun 29, 2024

View reviewed changes

tgazagnes merged commit dfaa73c into staging Jul 6, 2024
1 check passed

tgazagnes deleted the 22-onglet-data-améliorations branch July 6, 2024 16:13

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

22 onglet data ameliorations #29

22 onglet data ameliorations #29

tgazagnes commented Jun 20, 2024

tgazagnes commented Jun 20, 2024

KyllianBeguin commented Jun 20, 2024

tgazagnes commented Jun 20, 2024

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024

KyllianBeguin commented Jun 21, 2024

KyllianBeguin left a comment

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024 •

edited

Loading

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024 •

edited

Loading

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024

This comment was marked as outdated.

KyllianBeguin Jun 21, 2024

tgazagnes Jun 25, 2024

KyllianBeguin left a comment

KyllianBeguin Jun 29, 2024

		@@ -432,25 +432,30 @@ def calculate_and_display_metrics(data, indicator_col1, indicator_col2, indicato
		def couleur_milieu(type):

22 onglet data ameliorations #29

22 onglet data ameliorations #29

Conversation

tgazagnes commented Jun 20, 2024

tgazagnes commented Jun 20, 2024

KyllianBeguin commented Jun 20, 2024

tgazagnes commented Jun 20, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KyllianBeguin commented Jun 21, 2024

KyllianBeguin left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tgazagnes Jun 25, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tgazagnes Jun 25, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

This comment was marked as outdated.

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KyllianBeguin left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tgazagnes Jun 25, 2024 •

edited

Loading

tgazagnes Jun 25, 2024 •

edited

Loading