eduaguilera/whep

feed_intake_build: dead columns computed and joined in .build_feed_demand

Aberta

#222 aberto em 22 de jul. de 2026

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)R (3 forks)auto 404
area:livestockclean codegood first issuehelp wantedmechanicalno-data-neededpriority:low

Métricas do repositório

Stars
 (1 estrela)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Inspecione o código em R/feed intake build.R (linhas 204-209) e R/feed intake redistribute.R (linhas 398-401) para confirmar as colunas não utilizadas e o join de lookup. Remova as colunas mortas e o join e execute os testes existentes para garantir que nada quebre.
Pilha de tecnologia
r
Domain
data
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80