-
Notifications
You must be signed in to change notification settings - Fork 11
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
reorganize config file inputs #124
Comments
I like the idea of keeping the config file as a json file as this kind of file is easy to understand, is easy to edit, and easy to host and transmit across the internet. I think we should keep config files but have different kinds of config files depending on the task they are being used for. Here are a couple of ideas:
I’m not sure what you mean by ‘dataset’? |
If it helps plan this, right now we have the following mandatory training params (the program should be modified so it exits if not available):
Mandatory training params for ResUnet only (i.e. not needed by segformer models):
Optional training params:
Mandatory data params (or should these all be optional, defaulting to certain values?):
Optional data params
Optional inference parameters:
Optional general params:
|
One idea I had was to write a simple utility to generate a config file. User would provide at least
and the program could fill the remaining params with default values.... the user could then edit that file The program could be called |
I think I will go ahead and make this config generator tool, like I described above, in the next version |
in the long term I think having 46+ possible config items, some of which are mandatory and some of which are optional, is a clunky way to organize this. We either:
What do you say @2320sharon and @venuswku ?
Originally posted by @dbuscombe-usgs in #122 (comment)
The text was updated successfully, but these errors were encountered: