eduaguilera/whep

footprint_balance: compare_footprint_methods conflates NA-valued footprints with zero

Ouverte

#212 ouverte le 22 juil. 2026

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)R (3 forks)auto 404
area:footprintbuggood first issuehelp wantedmechanicalno-data-neededpriority:low

Métriques du dépôt

Stars
 (1 étoile)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Inspectez la fonction compare footprint methods() dans R/footprint balance.R. Concentrez vous sur la façon dont les valeurs NA traversent sum() et tidyr::replace na(), et réfléchissez à la manière de distinguer l'absence par manque de véritables footprints NA.
Stack technique
r
Domaine
data
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitR
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75