Improving implementation o movePeopleToEmptyHouse

[AtiyahElsheikh]

Within

function movePeopleToEmptyHouse!(people, dmax, allHouses, allTowns=Town[])

In module Simulate* allocate.jl in version 0.2 , argument dmax is of type symbol and an if else statements determines the dispatching to right implementation according to the dmax value (:near, :far, :hear).

An improved realization via multiple dispatch is realizable with traits.

[AtiyahElsheikh]

Additionally, current internal call to this function create temporary arrays for each function call leading to potential performance drawback.

[AtiyahElsheikh]

Also, in many related internal functions return two types, either a house or nothing. It is worth to examine if returning only one type improves performance.

[AtiyahElsheikh]

Internally, temporary arrays of references to empty houses are excessively created.

[AtiyahElsheikh]

• Syntactical error has been identified and removed. Simulation was running without any complains.
• Moreover, it seems there are more houses needed than there are
• moving to a house in the same town was integrated in moving to a house in an adjacent town. To follow the precise formulation of the model, the probability of moving to a house in the same town should equal 1/3 rather than 1/9

[AtiyahElsheikh]

Improvements:

• set of towns need not to be constant throughout the simulation
• the set of adjacent towns include only those with density > 0