eduaguilera/whep

footprint_uncertainty: propagate_fp_uncertainty accepts malformed probs unvalidated

Offen

#215 geöffnet am 22.07.2026

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)R (3 Forks)auto 404
area:footprintbuggood first issuehelp wantedmechanicalno-data-neededpriority:low

Repository-Metriken

Stars
 (1 Stern)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die Implementierung von `propagate fp uncertainty` in `R/footprint uncertainty.R`, verstehe, wie `options$probs` verwendet wird, und füge Validierungen für Länge, Bereich [0,1] und Sortierung hinzu. Schreibe Tests für ungültige Eingaben.
Tech Stack
r
Domain
data
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80