UAL-RE / LD-Cool-P

Python tool to enable data curation
MIT License
4 stars 5 forks source link

Switch to using JSON for configuration settings #243

Open astrochun opened 3 years ago

astrochun commented 3 years ago

Summary

In the past, we used a config INI file and configparser to parse the configuration. This was identical to how we did it with ReQUIAM. Over time, the complexity of configuration for LD-Cool-P grew. For example, handling multiple Deposit Agreement survey required a string version of a list that was interpreted via ast.literal. To simplify this, I recommend moving toward using a config.json file following a JSON schema. This will allow for long-term support of more complex configuration settings. It will also be easy to load the configuration using the built-in json library.

Objectives

A simplification of the configuration. While this is a change to the config file, we should ensure that it's backwards compatible. Namely that both .ini and .json files can be imported.

Proposal

Testing notes

See above todo action.

Additional notes

Implemented in: TBD