Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.Sign up
Small tweak to hierarchy file reader - patch for pre-AP simulations #80
This is described by the discussion in Issue #79 , and implements the fix described there. It fixes a bug where the recently-accepted active particle PR (PR #57 ) breaks restarts from simulations that have particles but predate the active particle PR merge (which is to say, almost all Enzo simulations ever).
brittonsmith left a comment •
I think this whole block can be refactored to be a bit more straightforward. At minimum, the train of comments here is pretty difficult to follow. Perhaps it could be simplified to something like, "If NumberOfActiveParticles line appears, the next two lines need to be read in but are currently unused."