This PR aims to fix issue #2 by refactoring the 'main' execution code of the model.
This includes several changes:
Replacing the earlier system of setting 'Options' through global name space variable definitions with a set of dictionaries that can be combined and passed.
Modularizing the code so that the 'main' functionality is called using a function (main()) rather than by executing the python file with exec()
Splitting out the CSTW Agent and Market definitions into a separate module, and using multiple inheritance to efficiently avoid the (very weird) variable superclass pointer.
removing do_all.py, which is not feasibly executed or tested.
This seems to be working well and is passing the tests I made earlier for do_min.py results.
I wonder if @alanlujan91 would be interested in reviewing before I merge it.
There is also an issue which is that do_mid is not currently working in master: #12
This PR aims to fix issue #2 by refactoring the 'main' execution code of the model.
This includes several changes:
main()) rather than by executing the python file withexec()do_all.py, which is not feasibly executed or tested.This seems to be working well and is passing the tests I made earlier for
do_min.pyresults.I wonder if @alanlujan91 would be interested in reviewing before I merge it.
There is also an issue which is that
do_midis not currently working in master: #12