search

MRC-CSO-SPHSU / LoneParentsModel.jl

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

Dead Person type #135

Closed AtiyahElsheikh closed 1 year ago

AtiyahElsheikh commented 1 year ago

Seems there could be a need to (optionally) store dead people (e.g. for statistical purposes). Therefore it could make sense to introduce a new immutable type for a deadPerson, e.g. 

struct DeadPerson 
  person:: Person 
  deathDate :: DateType   
end
AtiyahElsheikh commented 1 year ago

Optionally, because for performance and memory consumption reasons, simulations without such a need can be made enabled.

AtiyahElsheikh commented 1 year ago

Just saw, related to #116

mhinsch commented 1 year ago

We do have the latest age for each agent, so instead of creating a new type I'd rather store the birth date in the basic info (then we could infer death date).

AtiyahElsheikh commented 1 year ago

Just came to my thought. This is the idea of having a new type, otherwise, why to store additional data we won't need them in all possible simulation studies.

mhinsch commented 1 year ago

Either way, I think that's a problem that can be solved when it occurs.