Motivation / Current Behaviour
The default values for AgegroupGotoSchool/Work parameters are unsafe for small numbers of age groups. There could be wrong memory accesses if the age groups in the default values don't exist. They are also not great defaults for e.g. a large number of age groups. Usage is awkward, since all age groups have to be set to false before setting the desired age groups.
Enhancement description
Set a better default for the parameters (probably all age groups false).
Fix unit tests and benchmarks by explicitly setting the parameters.
Maybe CustomIndexArray can be extended to allow for a convenient syntax to set multiple entries to the same value, e.g. parameters.get<AgegroupGotoSchool>()[{Agegroup(1), Agegroup(2)}] = true;
Additional context
No response
Checklist
Motivation / Current Behaviour
The default values for
AgegroupGotoSchool/Workparameters are unsafe for small numbers of age groups. There could be wrong memory accesses if the age groups in the default values don't exist. They are also not great defaults for e.g. a large number of age groups. Usage is awkward, since all age groups have to be set to false before setting the desired age groups.Enhancement description
Set a better default for the parameters (probably all age groups false).
Fix unit tests and benchmarks by explicitly setting the parameters.
Maybe CustomIndexArray can be extended to allow for a convenient syntax to set multiple entries to the same value, e.g.
parameters.get<AgegroupGotoSchool>()[{Agegroup(1), Agegroup(2)}] = true;Additional context
No response
Checklist