eduaguilera/whep

footprint_balance: compare_footprint_methods conflates NA-valued footprints with zero

Aberta

#212 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

compare_footprint_methods() (R/footprint_balance.R:104-121) uses sum(value) with no na.rm, so a cell containing an NA sums to NA; the subsequent tidyr::replace_na(value_a, 0) — intended to fill cells absent from one method — silently rewrites that NA to 0.

Why it matters

A genuine-but-NA footprint is reported as "zero footprint", fabricating a spurious abs_diff equal to the other method's full value.

Suggested fix / check

Distinguish "absent from this method" from "present but NA" before replacing with 0.

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
Inspecione a função compare footprint methods() em R/footprint balance.R. Concentre se em como os valores NA passam por sum() e tidyr::replace na(), e considere como distinguir ausência por falta de verdadeiras pegadas NA.
Pilha de tecnologia
r
Domain
data
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
GitR
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75