eduaguilera/whep

calculate_lmdi: period_targets computed but never surfaced (output_format = "total" dead path)

Ouverte

#248 ouverte le 22 juil. 2026

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)R (3 forks)auto 404
area:lmdi-decompclean codegood 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

.lmdi_build_result (R/calculate_lmdi.R:1082-1099) builds period_target_df, .lmdi_process_all_groups (:1116-1183) collects it into all_results$period_targets, and it is returned — but .lmdi_format_output (:1206-1236) reads only all_results$results and never touches period_targets in either the "clean" or "total" branch.

Why it matters

Pure dead code; the "total" format presumably intended to surface this period-level summary (t0/t_t/target_initial/target_final/total_change).

Suggested fix / check

Either expose period_targets in the "total" output or remove the unused plumbing.

From deep static audit (2026-07-23), round 4, parallel subsystem review. Deduplicated against open+closed issues. Findings confirmed by running the functions unless stated.

Guide contributeur

Direction de recherche
Examinez le fichier R calculate lmdi.R, en vous concentrant sur .lmdi build result, .lmdi process all groups et .lmdi format output pour comprendre pourquoi period targets n'est pas inclus dans la sortie 'total'. Vérifiez les tests existants et la documentation utilisateur pour le comportement attendu.
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