AllenNeuralDynamics / AllenNeuralDynamics/aind-metadata-extractor

Consider using the pydantic-settings package

Open
#4 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
0
Forks
1
PR merge metrics
No merged PRs in 30d

Description

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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.