Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved configuration parsing #758

Open
sarthakpati opened this issue Dec 20, 2023 · 5 comments
Open

Improved configuration parsing #758

sarthakpati opened this issue Dec 20, 2023 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@sarthakpati
Copy link
Collaborator

sarthakpati commented Dec 20, 2023

Is your feature request related to a problem? Please describe.
Currently, the configuration parsing is done by a single submodule [ref], which is creates issues with maintenance as more functionality is introduced.

Describe the solution you'd like
Using a configuration validation mechanism that is more standardized would make the developer and user experience much better. Some examples are (and this is by no means an exhaustive list):

Package Documentation Example
Cerberus https://docs.python-cerberus.org/ https://stackoverflow.com/a/46626418
PyDantic https://docs.pydantic.dev/latest/ https://stackoverflow.com/a/61021183
Marshmallow https://marshmallow.readthedocs.io/en/stable/ https://stackoverflow.com/a/63739747

Describe alternatives you've considered
N.A.

Additional context
It would make sense to eventually move GaNDLF's various functionalities to something like Hydra, and this could potentially be a good starting point.

@sarthakpati sarthakpati added the enhancement New feature or request label Dec 20, 2023
@vedik2002
Copy link

Hi @sarthakpati can you please assign this issue to me.

@sarthakpati
Copy link
Collaborator Author

Thanks, done.

Copy link
Contributor

github-actions bot commented Mar 7, 2024

Stale issue message

Copy link
Contributor

github-actions bot commented May 9, 2024

Stale issue message

@sarthakpati
Copy link
Collaborator Author

Hi @vedik2002, are you still working on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants