search

educational-technology-collective / morf

The MOOC Replication Framework (MORF)
MIT License
16 stars 7 forks source link

Make configuration class #6

Closed jpgard closed 6 years ago

jpgard commented 6 years ago

No reason to pass around so many parameters or read them from config file. There should be a morfJobConfig class (or something similar) which contains everything needed to run the job. Then, there can be some kind of constructor function (very similar to get_config_properties()) which will read the config file and return a single morfJobConfig instance.

jpgard commented 6 years ago

resolved with merge of jobconfig-train branch onto master.