I've tested the second commit - resuming from the latest timestep in a number of different configurations here and not had any issues with it. It seems to do the correct thing and restart the simulation when required.
I added testing these two new properties in the LESStep demo. It shouldn't have any material impact and the tests still pass.
I haven't explicitly added a test that stops and resumes a solve as it seems tricky to avoid any timing issues and would add a lot of complexity.
I've tested the second commit - resuming from the latest timestep in a number of different configurations here and not had any issues with it. It seems to do the correct thing and restart the simulation when required.
I added testing these two new properties in the
LESStep
demo. It shouldn't have any material impact and the tests still pass.I haven't explicitly added a test that stops and resumes a solve as it seems tricky to avoid any timing issues and would add a lot of complexity.