resgroup/wind-up

allow more config to be unspecified in yaml

Open

#68 geöffnet am 18. Sept. 2025

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (11 Forks)auto 404
good first issue

Repository-Metriken

Stars
 (28 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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)

Contributor Guide

Research-Richtung
Finde das pydantic Konfigurationsmodell im Code. Identifiziere die drei Felder: years offset for pre period, ws bin width, years for detrend. Ändere das Modell, um sie optional zu machen (Optional Typ) und setze die Standardwerte (1, 1, 3). Aktualisiere alle Tests, die darauf angewiesen sind, dass diese Felder erforderlich sind.
Tech Stack
python
Domain
clidata
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Python
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80