Skip to content
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

Change around logic for creating time files #231

Merged
merged 1 commit into from
Aug 12, 2018

Conversation

mandli
Copy link
Member

@mandli mandli commented Aug 10, 2018

Basically move the creation of the 'timing.csv' files to amr*.f90. Also removes the 1D code from the 2D stuff given that we have an explicit 1D code now (supercedes #223).

@rjleveque
Copy link
Member

This seems fine. It still gives some garbage at the first output time when doing a restart, but that's not so easy to fix and at least all the timing data is preserved and the user can edit timing.csv to clean it up if desired.

@rjleveque rjleveque merged commit 1d6d2a5 into clawpack:master Aug 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants