InstituteforDiseaseModeling / laser

Light Agent Spatial modeling for ERadication
MIT License
2 stars 5 forks source link

Transpose array allocation in add_vector_property() (in population.py) #38

Open clorton opened 2 months ago

clorton commented 2 months ago

Probably doesn't have any measurable performance impact, but the current specification of dimensions in array allocation means that the most likely access pattern does not access contiguous locations in memory.

Also, property[tick, :] might be a little more sensible than property[:, tick] from an API perspective.

This would be a breaking change.

krosenfeld-IDM commented 2 months ago

hi @clorton