baliuzeger / sinking.jl Goto Github PK

Description

By the macro @time, current code has unnecessary memory allocation. There causes may be:

1. In Seat of Population, the field agent uses abstract type Agent.
2. the propagating signals are collected under abstract type Signal 1st and then let the receiving agents accept later.

Solution

cause #1

For each type of agent, just make a concrete type dictionary under Population.

cause #2

Make an interface update, and let every type of agents which have states implement update(::Agent, ::States). Let act take an updater function updater(::Agent, ::States) as an argument (dependency injection). The updater use lock to prevent data racing 1st and then trigger update, and it's locks are defined in async_simulate.

Also, let act take another function push_signal(::address, ::a_signal), which locks the receiver of the signal, get the receiver agent, and then call accept(::an_agent, ::a_signal) on it.

Reference

https://docs.julialang.org/en/v1/manual/performance-tips/