entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 11 Releases Wiki Activity

dataviz: add pie percent chart type (#71666) #256

Merged
vdeniaud merged 1 commits from wip/71666-dataviz-ajouter-un-type-de-graph into main 2024-04-29 09:43:56 +02:00
Conversation 0 Commits 1 Files Changed 3 +42 -4
vdeniaud commented 2024-04-24 17:30:06 +02:00
Owner
Copy Link
No description provided.
vdeniaud added 1 commit 2024-04-24 17:30:07 +02:00
gitea/combo/pipeline/head This commit looks good Details
47e2df1bbd dataviz: add pie percent chart type (#71666)
vdeniaud force-pushed wip/71666-dataviz-ajouter-un-type-de-graph from 47e2df1bbd to 2b29818899 2024-04-24 17:40:11 +02:00 Compare
vdeniaud changed title from WIP: dataviz: add pie percent chart type (#71666) to dataviz: add pie percent chart type (#71666) 2024-04-24 17:42:04 +02:00
yweber reviewed 2024-04-25 10:27:47 +02:00
combo/apps/dataviz/models.py Outdated
@ -746,0 +759,4 @@
factor = 100 / sum_values
for i, values in enumerate(zip(*series_data)):
for values in series_data:
if values[i] is not None:
yweber commented 2024-04-25 10:27:47 +02:00
Owner
Copy Link

Je ne comprend pas trop l'avantage de l'utilisation du zip et l'écrasement de values par le deuxième for ici.

Est-ce que ça serait pas équivalent tout en étant plus simple/lisible de faire deux for imbriqués "classique" du genre :

for serie in series_data:
    for i, value in enumerate(serie):
        if value is not None:
            serie[i] = round(value * factor)
Je ne comprend pas trop l'avantage de l'utilisation du zip et l'écrasement de values par le deuxième for ici. Est-ce que ça serait pas équivalent tout en étant plus simple/lisible de faire deux for imbriqués "classique" du genre : ```python for serie in series_data: for i, value in enumerate(serie): if value is not None: serie[i] = round(value * factor) ```
vdeniaud commented 2024-04-25 10:53:06 +02:00
Author
Owner
Copy Link

Clairement, j'ai juste copié/collé la boucle de la fonction plus haut sans réfléchir alors qu'ici il n'y a pas besoin de se compliquer la vie. C'est appliqué, merci !

Clairement, j'ai juste copié/collé la boucle de la fonction plus haut sans réfléchir alors qu'ici il n'y a pas besoin de se compliquer la vie. C'est appliqué, merci !
yweber marked this conversation as resolved
vdeniaud force-pushed wip/71666-dataviz-ajouter-un-type-de-graph from 2b29818899 to 32c912b978 2024-04-25 10:51:42 +02:00 Compare
yweber reviewed 2024-04-25 11:03:59 +02:00
yweber left a comment
Owner
Copy Link

Une dernière petite question et c'est bon pour moi.

Une dernière petite question et c'est bon pour moi.
combo/apps/dataviz/models.py
@ -719,3 +727,3 @@
@staticmethod
def get_value_formatter(unit, measure='duration'):
def get_value_formatter(unit=None, measure='duration'):
yweber commented 2024-04-25 11:01:31 +02:00
Owner
Copy Link

J'ai un peu de mal à saisir le but de l'ajout de la valeur None par défaut, elle ne me semble jamais utilisée.

D'autant que si je suit bien le reste de la fonction, avec unit=None la fonction va renvoyer None (et j'imagine que c'est pas un formatter valide ? ).

J'ai un peu de mal à saisir le but de l'ajout de la valeur None par défaut, elle ne me semble jamais utilisée. D'autant que si je suit bien le reste de la fonction, avec unit=None la fonction va renvoyer None (et j'imagine que c'est pas un formatter valide ? ).
vdeniaud commented 2024-04-25 11:36:01 +02:00
Author
Owner
Copy Link

Je mets None par défaut simplement pour rendre le paramètre optionnel. Effectivement appeler la méthode sans paramètres n'a pas de sens mais puisqu'on ne le fera pas ça ne pose pas de soucis :)

(alternativement plus haut j'aurais pu faire chart.config.value_formatter = self.get_value_formatter('xxx', measure='percent') mais c'est un peu moche)

((après cette partie du code est schyzo car elle est appelée d'un côté par les stats legacy produites par la brique bijoe, et également ici par les stats nouvelle génération : un jour on pourra dropper le support de bijoe et ce paramètre unit disparaîtra))

Je mets None par défaut simplement pour rendre le paramètre optionnel. Effectivement appeler la méthode sans paramètres n'a pas de sens mais puisqu'on ne le fera pas ça ne pose pas de soucis :) (alternativement plus haut j'aurais pu faire `chart.config.value_formatter = self.get_value_formatter('xxx', measure='percent')` mais c'est un peu moche) ((après cette partie du code est schyzo car elle est appelée d'un côté par les stats legacy produites par la brique bijoe, et également ici par les stats nouvelle génération : un jour on pourra dropper le support de bijoe et ce paramètre `unit` disparaîtra))
yweber marked this conversation as resolved
yweber approved these changes 2024-04-25 12:05:15 +02:00
vdeniaud merged commit 7961c5544f into main 2024-04-29 09:43:56 +02:00
vdeniaud deleted branch wip/71666-dataviz-ajouter-un-type-de-graph 2024-04-29 09:43:56 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
yweber
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/combo#256
No description provided.
Delete Branch "wip/71666-dataviz-ajouter-un-type-de-graph"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?