eduaguilera/whep

footprint_uncertainty: propagate_fp_uncertainty accepts malformed probs unvalidated

Aperta

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

propagate_fp_uncertainty() (R/footprint_uncertainty.R:154-169, 185-195) merges options$probs with no length/range check. probs = c(0.05, 0.95) errors only after all n draws are computed (probs[3] → NA in quantile), and unordered probs silently produce q_low > q_high.

Suggested fix / check

Validate probs (length, range [0,1], ordering) at entry before drawing.

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
Esamina l'implementazione di `propagate fp uncertainty` in `R/footprint uncertainty.R`, comprendi come viene utilizzato `options$probs` e aggiungi la convalida per lunghezza, intervallo [0,1] e ordinamento. Scrivi test per input non validi.
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
RGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80