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

Copy gauge output from Fortran rather than rewrite from Python #102

Merged
merged 2 commits into from
Sep 23, 2016

Conversation

mandli
Copy link
Member

@mandli mandli commented Jun 6, 2016

Addresses #101.

Needs to point to temporary path rather than test path
@mandli mandli added this to the 5.4 milestone Aug 30, 2016
@mandli mandli merged commit 6fb8d25 into clawpack:master Sep 23, 2016
@rjleveque
Copy link
Member

@mandli: Should all the regression data in other repos now be regenerated with this change?

@mandli
Copy link
Member Author

mandli commented Sep 25, 2016

Yeah, probably. I think the Python and Fortran formats are still slightly different so it might be best to do so.

rjleveque added a commit to rjleveque/amrclaw that referenced this pull request Oct 9, 2016
rjleveque added a commit to rjleveque/geoclaw that referenced this pull request Oct 9, 2016
…me data, different format following clawpack/clawutil#102. Also added claw_git_status files
rjleveque added a commit to rjleveque/geoclaw that referenced this pull request Oct 9, 2016
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