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

Reordered 3d Classic code #20

Merged
merged 9 commits into from
Jul 30, 2013
Merged

Reordered 3d Classic code #20

merged 9 commits into from
Jul 30, 2013

Conversation

gradylemoine
Copy link
Contributor

Reordered 3D Classic code, based as much as possible on 3D PyClaw reordered Fortran for commonality's sake. Adds dynamic allocation, removes maxmx etc. This pull request also includes a variable-coefficient acoustics example, which uses the solver from the Riemann repo and the updated data.py from my clawutil pull request. I've tested it against the equivalent Clawpack 4.3 problem, and the final output files are identical.

@mandli
Copy link
Member

mandli commented Jul 25, 2013

Need consistent repository paths, the Makefile in the example refers to RIEMANN and uses CLAW for the Makefile.common. Moving forward we should use only $CLAW but that has not been implemented yet.

@mandli
Copy link
Member

mandli commented Jul 25, 2013

Also, need to update to use the new parameters in clawdata.py. See clawpack/clawutil#44 and clawpack/clawutil#43.

@gradylemoine
Copy link
Contributor Author

Updated to use new clawdata.py in example's setrun.py and in claw3ez.f. Also switched to new dynamic allocation setup in claw3ez and driver.f, as discussed on claw-dev.

@mandli
Copy link
Member

mandli commented Jul 30, 2013

I am adding classic as a subrepository to the clawpack-org repository first and then I will pull this in.

@mandli
Copy link
Member

mandli commented Jul 30, 2013

Works for me.

mandli added a commit that referenced this pull request Jul 30, 2013
@mandli mandli merged commit e5e2578 into clawpack:master Jul 30, 2013
@gradylemoine gradylemoine deleted the reordered-3d branch July 30, 2013 17:40
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