search

MRC-CSO-SPHSU / LoneParentsModel.jl

An initial implementation of an ABM for social and child care
0 stars 4 forks source link

Demography Data #142

Closed AtiyahElsheikh closed 1 year ago

AtiyahElsheikh commented 1 year ago

As mentioned in a comment within mainHelpers.jl, demographydata.jl needs to be placed within a module.

I see two options:

Although currently not that significant amount of code within demographydata.jl, it makes sense to have a module for Data

AtiyahElsheikh commented 1 year ago

repeated in #147

mhinsch commented 1 year ago

As mentioned in a comment within mainHelpers.jl, demographydata.jl needs to be placed within a module.

I see two options:

* as it is related to ParamTypes, it fits there

* or another module for Data

Although currently not that significant amount of code within demographydata.jl, it makes sense to have a module for Data

Why does it need to be placed within a module?