eduaguilera/whep

footprint_balance: compare_footprint_methods conflates NA-valued footprints with zero

Aperta

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

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.

Guida contributor

Direzione di ricerca
Ispeziona la funzione compare footprint methods() in R/footprint balance.R. Concentrati su come i valori NA fluiscono attraverso sum() e tidyr::replace na(), e considera come distinguere l'assenza per mancanza da veri footprint NA.
Tech stack
r
Dominio
data
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
GitR
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75