eduaguilera/whep

footprint_sankey: a genuine category literally named "Other" is merged into the grouped bucket

Ouverte

#214 ouverte le 22 juil. 2026

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)R (3 forks)auto 404
area:footprintbuggood first issuehelp wantedmechanicalno-data-neededpriority:low

Métriques du dépôt

Stars
 (1 étoile)
Métriques de merge PR
 (Métriques PR en attente)

Description

What

.sankey_group_small_nodes() (R/footprint_sankey.R:442-455, mirrored in JS at :673) excludes any node equal to other_label from the keep-set, so a real item/area label "Other" (which occurs in FAO-style nomenclatures, e.g. "Meat, Other") is silently pooled with truncated small nodes, misattributing its flow.

Suggested fix / check

Key the small-node bucket on a sentinel that cannot collide with a real label, or track group membership separately from the display label.

From deep static audit (2026-07-22), parallel subsystem review pass. Deduplicated against then-open issues. Not verified by running unless stated.

Guide contributeur

Direction de recherche
Examinez la fonction .sankey group small nodes() dans R/footprint sankey.R et son équivalent JS pour comprendre comment 'other label' est utilisé pour regrouper les petits nœuds. Identifiez pourquoi un vrai libellé 'Other' est incorrectement traité comme marqueur de groupe. Examinez la logique du code et envisagez des tests couvrant un libellé nommé 'Other'. Explorez des solutions qui utilisent une valeur sentinelle ou qui suivent le regroupement séparément.
Stack technique
javascriptr
Domaine
data visualization
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitRJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75