eduaguilera/whep
feed_intake_build: dead columns computed and joined in .build_feed_demand
開放
#222 建立於 2026年7月22日
area:livestockclean codegood first issuehelp wantedmechanicalno-data-neededpriority:low
倉庫指標
- 星標
- (1 顆星)
- PR 合併指標
- (PR 指標待抓取)
描述
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.