eduaguilera/whep

compare_global_whep.R: multi-column comparison verdict reflects only the last value column

Aberta

#263 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:cbsbuggood first issuehelp wantedmechanicalno-data-neededpriority:lowtesting

Métricas do repositório

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

Description

What

In compare_global_whep.R:288-296, n_differ is reassigned inside for (vc in val_cols_present) and then used after the loop for the IDENTICAL vs DIFFERENCES FOUND verdict. For the Processing Coefficients comparison (6 value columns), the verdict is decided only by the last column (final_value_processed).

Why it matters

If an earlier column diverges but the last matches within tolerance (and keys fully overlap), the script prints "IDENTICAL (within tolerance)". Per-column diffs are still printed above, so it's a summary-level false-confidence issue. (Primary and CBS have a single value column, so they're unaffected.)

Suggested fix / check

Accumulate n_differ across all value columns (e.g. any/sum) before deciding the verdict.

From deep static audit (2026-07-23), round 7, inst/scripts validation/diagnostic scripts. Deduplicated against open+closed issues. Not verified by running unless stated. Related to the false-confidence-tests concern in #177.

Guia do colaborador

Direção de pesquisa
Examine a lógica de comparação de múltiplas colunas no script compare global whep.R, focando em como a variável n differ é atualizada dentro do loop. Trace como o veredito final depende da última coluna e pense em quais alterações agregariam corretamente as diferenças em todas as colunas.
Pilha de tecnologia
r
Domain
data
Tipo Issue
Bug
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.
75