Development #37903
dataviz : faire apparaître les totaux sur les tableaux
0%
Description
Tri croisé classique
Dans la visualisation dans Bi-joe, on a une colonne total
Cette colonne totale n'apparaît pas dans Bi-Joe
exemple
vue bi-joe :
https://statistiques-grand-chambery.test.entrouvert.org/visualization/26/
vue combo : (Nb de demandes par sous-types et par mois)
https://agents-grand-chambery.test.entrouvert.org/stats/stats-signalements-ville-de-chambery/stats-signalement-eclairage-public-ville/
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Projet changé de BiJoe à Combo
- Sujet changé de Faire apparaître les totaux sur les tableaux dans combo comme dans Bi-joe à dataviz : faire apparaître les totaux sur les tableaux
C'est pygal qui fait le rendu des tableaux et qui n'a pas cette option, ça passerait, dans le cas d'un rendu tableau, par le calcul dans combo et l'ajout de colonne/ligne et des résultats dans la matrice.
Mis à jour par Frédéric Péters il y a plus de 4 ans
... et qui n'a pas cette option ...
À regarder de plus près, il y a cette option, mais elle ajoute des totaux pour les deux axes, même quand il y a une seule série, ce qui n'est pas terrible; et par rapport à l'approche consistant à ajouter colonne/ligne dans la matrice que je donnais, je me demande maintenant si le plus simple n'est pas de prendre le rendu "tableau" à charge de Combo.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
Pas de commit dans pygal depuis plus d'un an, personne ne regarde les PR, dead end donc.
Du coup j'ai commencé à regarder et pour avoir les totaux qui s'affichent au même moment que dans bijoe il semble qu'il suffise de faire
ctx['table'] = chart.render_table( transpose=bool(chart.axis_count == 2), + total=bool(chart.axis_count > 1), )
Ce qui me paraît bien plus simple que de « prendre le rendu "tableau" à charge de Combo ». Je rate quelque chose ?
Mis à jour par Frédéric Péters il y a plus de 4 ans
Pas retesté mais de mon commentaire précédent ça ajoute des totaux pour les deux axes, ex:
----------------------- | | Totaux Pommme | 12 | 12 Poire | 5 | 5 Pêche | 8 | 8 Abricot | 11 | 11 ----------------------- Totaux | 36 | 36
alors que cette colonne à droite est inutile.
Aussi, par rapport à pygal même, on peut se dire qu'on utilise qu'on aura d'autres modifs qu'on voudra y voir, et travailler un fork, reprendre à un moment la maintenance, etc.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
C'est pour cela que j'ai proposé une condition pour ne pas afficher les totaux si il n'y a qu'un axe. Après en écrivant que l'un est inutile tu sous-entends qu'on devrait afficher l'autre, mais ce n'est pas ce que fait bijoe. Donc je pense que ce patch d'une ligne convient, je vais écrire des tests.
Mis à jour par Frédéric Péters il y a plus de 4 ans
C'est pour cela que j'ai proposé une condition pour ne pas afficher les totaux si il n'y a qu'un axe.
Mais ça retirerait la ligne "total" intéressante.
----------------------- | | Totaux (on Pommme | 12 | 12 ne Poire | 5 | 5 veut Pêche | 8 | 8 pas Abricot | 11 | 11 ceux-ci) ----------------------- Totaux | 36 (il faut celui-ci)
mais ce n'est pas ce que fait bijoe.
Ce n'est pas un argument, combo cherche à mieux travailler l'affichage.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-dataviz-display-total-in-tables-37903.patch 0001-dataviz-display-total-in-tables-37903.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
OK, il reste le cas où bijoe n'affiche pas les totaux à raison, genre sur des pourcentages et des moyennes de durée. Pour cela il se base sur la requête sql, du coup il n'y a pas vraiment de critère équivalent côté combo, je me suis basé sur la clé 'measure' par défaut.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
x_labels.append('Total')
Manque l'appel à gettext.
À part ça, ok, même si à un moment il faudra à mon avis reprendre pour permettre classe/style partiuclier pour les totaux.
Mis à jour par Valentin Deniaud il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 83a8517d282f28a4c00642aafe5e44b9b32c0c51 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Jan 8 11:36:26 2020 +0100 dataviz: display total in tables (#37903)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
dataviz: display total in tables (#37903)