resgroup/wind-up

allow more config to be unspecified in yaml

Open

#68 aberto em 18 de set. de 2025

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (11 forks)auto 404
good first issue

Métricas do repositório

Stars
 (28 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

I was setting up a northing analysis today and tried to run with a yaml missing the below three settings, thinking they would just use suitable defaults. The analysis failed with a pydantic validation error unless these settings were populated in my yaml. It would be nice to not have to specify these in every yaml because they are rarely set to a non-default value.

  • years_offset_for_pre_period (default should be 1)
  • ws_bin_width (default should be 1)
  • years_for_detrend (default should be 3)

Guia do colaborador

Direção de pesquisa
Localize o modelo de configuração pydantic no código. Identifique os três campos: years offset for pre period, ws bin width, years for detrend. Modifique o modelo para torná los opcionais (tipo Optional) e defina os valores padrão (1, 1, 3 respectivamente). Atualize quaisquer testes que dependem desses campos como obrigatórios.
Pilha de tecnologia
python
Domain
clidata
Tipo Issue
Funcionalidade
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Python
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80