AllenNeuralDynamics / AllenNeuralDynamics/aind-metadata-extractor

Consider using the pydantic-settings package

Aperta
#4 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Python
Stelle
0
Fork
1
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

**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.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.