search

mle-infrastructure / mle-toolbox

Lightweight Tool to Manage Distributed ML Experiments 🛠
https://mle-infrastructure.github.io/mle_toolbox/toolbox/
MIT License
3 stars 1 forks source link

Replace `cc` with `mle_config` & don't always load with each import #53

Closed RobertTLange closed 3 years ago

RobertTLange commented 3 years ago

Many of the imported files start with loading the cluster configuration from the .toml file:

from ..utils import load_mle_toolbox_config
# Import cluster credentials/settings - SGE or Slurm scheduling system
cc = load_mle_toolbox_config()

This is redundant since the config doesn't change during one mle <subcommand> execution. Instead I want to load it only once in the utils and then import the loaded config DotMap object instance. Also for better readability rename all cc to mle_config.

RobertTLange commented 3 years ago

Addressed in 7a53dec5b207ba514cefc19b7e2f6696a430a314