martius-lab / cluster_utils

https://cluster-utils.readthedocs.io/stable/
Other
12 stars 0 forks source link

Use argparse for reading arguments of hp_optimization/grid_search #103

Closed luator closed 5 months ago

luator commented 6 months ago

Use argparse instead of read_params_from_cmdline in the hp_optimization and grid_search scripts. This would give the following advantages:

(*) In addition, we could consider to also use argparse inside read_params_from_cmdline to improve readability of the code.

luator commented 6 months ago

I think we should tackle this one as a prerequisite for #81. I assume it will not easily be possible to auto-detect whether a config file is meant for smart_settings or omegaconf, so we will need a flag to indicate it (something like grid_search --smart-settings config.json). This would be much easier if switching to argparse first.