Fixed conduction timestep calculation.

#341 Merged at 152260b
Repository
Deleted repository
Branch
week-of-code (152260b5012e)
Repository
enzo-dev
Branch
week-of-code
Author
  1. Duncan Christie
Reviewers
Description

Within SetLevelTimeStep, a loop over grid calculates the conduction timestep for each grid. The results from the previous timestep calculation are never stored and are thus lost. This fix introduces a temporary variable and a call to min() to ensure that the minimum conduction timestep across grids is found.

Comments (2)

  1. Brian O'Shea

    nice catch, @dachrist . Nominally it's bad form to approve one's own pull request, but this is a small and simple change. I'll approve and merge.

  2. Duncan Christie author

    Ha! I was wondering why it gave me the option to approve my own pull request, and I pressed it out of curiosity. :-)