Refactoring of the time-step communication tasks
Compare changes
Significant re-factoring of the way the time-step sizes are being exchanged.
Same as !1455 (closed) but without the last batch of changes.
Summary:
engine_collect_end_of_step()
recurse.timestep
, timestep_limiter
, and timestep_sync
tasks all unlock that top-level task.engine_collect_end_of_step()
now only loops (via threadpool) over the local top-level cells. No recursion any more.tend
communication tasks that used to live at the super level are removed.engine_launch()
done every step to deal with the timestep limiter effect is removed (as it is now properly dealt with by the top-level task dependency)This should help speed up the smallest steps by reducing the level of the plateau we usually see in the "main sequence" plots.