GaNDLF icon indicating copy to clipboard operation
GaNDLF copied to clipboard

Improved configuration parsing

Open sarthakpati opened this issue 2 years ago • 13 comments

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 avatar Dec 20 '23 16:12 sarthakpati

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

vedik2002 avatar Jan 07 '24 05:01 vedik2002

Thanks, done.

sarthakpati avatar Jan 07 '24 11:01 sarthakpati

Stale issue message

github-actions[bot] avatar Mar 07 '24 19:03 github-actions[bot]

Stale issue message

github-actions[bot] avatar May 09 '24 19:05 github-actions[bot]