eduaguilera/whep

feed_intake_build: dead columns computed and joined in .build_feed_demand

Ouverte

#222 ouverte le 22 juil. 2026

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

In .build_feed_demand (R/feed_intake_build.R:204-209), demand_tot and demand_share are computed per (year, area_code, feed_type) and .feed_animal_type_lookup (graniv_grazers) is joined, but the sole consumer .build_feed_demand_codes (R/feed_intake_redistribute.R:398-401) reads only demand_aft. These are dead.

Why it matters

Harmless, but demand_share = demand_aft / (demand_tot * 1000) (with demand_tot itself pre-divided by 1000) is an error-prone expression a future reader could misuse.

Suggested fix / check

Remove the unused columns and lookup join.

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 le code dans R/feed intake build.R (lignes 204-209) et R/feed intake redistribute.R (lignes 398-401) pour confirmer les colonnes inutilisées et la jointure de recherche. Supprimez les colonnes mortes et la jointure, puis exécutez les tests existants pour vous assurer que rien ne casse.
Stack technique
r
Domaine
data
Type d'issue
Refactorisation
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
RGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80