-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better restarts #137
Better restarts #137
Conversation
…mpiler stops complaining about too small field widths
This failing because |
We need to update the documentation if we make this change, with some warnings that this isn't always the right thing to do. E.g. if the user restarts from a checkpoint file that wasn't at the final time from the previous run, or if several restarts are done from the same checkpoint file. We started some discussion of this in #39. Current documentation at http://www.clawpack.org/restart.html#output-files-after-a-restart |
@mjberger can you merge in the current master branch so that the tests can be run? |
Thinking more about this, I think appending to The way it's done in #144, the latest |
Your alternative approach is ok too although I think it requires more manual intervention. Inf the ordinary restart is bcause you want to run something longer, then appending is the right thing to do, instead of manually concatenating files. In the repeated case, you have to edit out the last lines from where you restart. My main concern is not to lose data, so either is ok, but simply overwriting isn't. |
Closing this since we decided not to do it this way. |
added capability to restart the gauge output instead of clobbering the existing file
(did same in geoclaw)
also fixed a few output field widths.