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

Need to a better way to configure poretitioner? #73

Open
kdoroschak opened this issue Dec 15, 2020 · 0 comments
Open

Need to a better way to configure poretitioner? #73

kdoroschak opened this issue Dec 15, 2020 · 0 comments
Assignees
Milestone

Comments

@kdoroschak
Copy link
Member

I didn't realize how painful dictionaries can be for configs. I'm currently just passing around a dictionary, but there's no guarantee that any particular values are defined (no validation), no guarantee that a variable has a specific type or format, and it's not easy to document the contents. I've been making up values as I go along with the intention of going back and writing definitions, but it's a mess.

We likely need some kind of config class, perhaps using configparser (since it's a default python library) to specify required & default values. I used yaml in the past, but that also has the same problems unless we add a wrapper to validate the config file and set default values.

Still thinking about this. Other configuration solutions welcome too.

@thequicksort thequicksort self-assigned this Jan 4, 2021
@kdoroschak kdoroschak added this to the 1.0.0 milestone Feb 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants