eduaguilera/whep

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

Aperta

#214 aperta il 22 lug 2026

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)R (3 fork)auto 404
area:footprintbuggood first issuehelp wantedmechanicalno-data-neededpriority:low

Metriche repository

Star
 (1 stella)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la funzione .sankey group small nodes() in R/footprint sankey.R e la sua controparte JS per capire come 'other label' viene usato per raggruppare piccoli nodi. Identifica perché un vero etichetta 'Other' viene erroneamente trattata come marcatore di gruppo. Rivedi la logica del codice e considera test che coprano un'etichetta chiamata 'Other'. Esplora soluzioni che usano un valore sentinella o che tracciano il raggruppamento separatamente.
Tech stack
javascriptr
Dominio
data visualization
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitRJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75