eduaguilera/whep

feed_intake_build: dead columns computed and joined in .build_feed_demand

Aperta

#222 aperta il 22 lug 2026

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)R (3 fork)auto 404
area:livestockclean codegood first issuehelp wantedmechanicalno-data-neededpriority:low

Metriche repository

Star
 (1 stella)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Ispeziona il codice in R/feed intake build.R (righe 204-209) e R/feed intake redistribute.R (righe 398-401) per confermare le colonne inutilizzate e il join di lookup. Rimuovi le colonne morte e il join, quindi esegui i test esistenti per assicurarti che nulla si rompa.
Tech stack
r
Dominio
data
Tipo issue
Refactoring
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