AllenNeuralDynamics / AllenNeuralDynamics/aind-metadata-extractor

Consider using the pydantic-settings package

Offen
#4 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
0
Forks
1
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

**Is your feature request related to a problem? Please describe.**
The JobSettings class imports both pydantic-settings and argparse

**Describe the solution you'd like**
The latest pydantic_settings.BaseSettings class has a convenient argument parsing functionality. We can also re-think how config files are parsed without the need of setting an explicit field.

**Describe alternatives you've considered**
We can leave things as they are, but the latest version might be easier to maintain long-term.

**Additional context**
Add any other context or screenshots about the feature request here.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.