eduaguilera/whep

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

Aberta

#214 aberto em 22 de jul. de 2026

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)R (3 forks)auto 404
area:footprintbuggood first issuehelp wantedmechanicalno-data-neededpriority:low

Métricas do repositório

Stars
 (1 estrela)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Examine a função .sankey group small nodes() no R/footprint sankey.R e seu espelho em JS para entender como 'other label' é usado para agrupar pequenos nós. Identifique por que um rótulo genuíno 'Other' é incorretamente tratado como marcador de grupo. Revise a lógica do código e considere testes que cubram um rótulo chamado 'Other'. Explore soluções que usem um valor sentinela ou rastreiem o agrupamento separadamente.
Pilha de tecnologia
javascriptr
Domain
data visualization
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitRJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75