eduaguilera/whep

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

Offen

#214 geöffnet am 22.07.2026

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)R (3 Forks)auto 404
area:footprintbuggood first issuehelp wantedmechanicalno-data-neededpriority:low

Repository-Metriken

Stars
 (1 Stern)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die Funktion .sankey group small nodes() in R/footprint sankey.R und ihr JS Pendant, um zu verstehen, wie 'other label' zum Gruppieren kleiner Knoten verwendet wird. Finden Sie heraus, warum ein echtes 'Other' Label fälschlicherweise als Gruppenmarker behandelt wird. Überprüfen Sie die Code Logik und ziehen Sie Tests in Betracht, die ein Label namens 'Other' abdecken. Erkunden Sie Lösungen, die einen Sentinel Wert verwenden oder die Gruppierung separat verfolgen.
Tech Stack
javascriptr
Domain
data visualization
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitRJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75