The way configs are handled in Melusine is not ideal:
A ConfigJsonReader class object is instanciated at the beggining of each module
The conf files are read multiple times
Using custom configurations is more complex than it should be
The csv format for the names file is not robust
Fixes #96
Type of change
Please delete options that are not relevant.
[ x] New feature (non-breaking change which adds functionality or feature that would cause existing functionality to not work as expected)
How Has This Been Tested?
[ x] Run pytest with the new configurations strategy
Test Configuration:
OS: Mac OS Catalina
Python version: 3.6
Melusine version: 2.3.1
Checklist:
[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ x] I have commented my code, particularly in hard-to-understand areas
[ x] I have made corresponding changes to the documentation
[ x] My changes generate no new warnings
[ x] New and existing unit tests pass locally with my changes
Description
The way configs are handled in Melusine is not ideal:
A ConfigJsonReader class object is instanciated at the beggining of each module The conf files are read multiple times Using custom configurations is more complex than it should be The csv format for the names file is not robust
Fixes #96
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Test Configuration:
Checklist: