serac icon indicating copy to clipboard operation
serac copied to clipboard

Restarts are running for full time step amount

Open white238 opened this issue 2 years ago • 0 comments

Cycle is loaded from previous run here:

https://github.com/LLNL/serac/blob/4c19ba966aeed4bc73b3ac5513fbd45eed370451/src/drivers/serac.cpp#L162

but the time step is not adjusted here:

https://github.com/LLNL/serac/blob/4c19ba966aeed4bc73b3ac5513fbd45eed370451/src/drivers/serac.cpp#L201-L203

This should also be something like this:

  // Initialize/set the time information
  double t;
  if (restart_cycle) {
    t = // grab time from datacoll based on given cycle;
  } else {
    t       = 0;
  }

This ends up running the following amount of steps on a restart:

(time step of cycle loaded) + full simulation time step amount = too much!

Thanks to @chapman39 for noticing!

white238 avatar Apr 12 '22 06:04 white238