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

Faster bound #173

Merged
merged 20 commits into from
Jan 10, 2016
Merged

Faster bound #173

merged 20 commits into from
Jan 10, 2016

Conversation

mjberger
Copy link
Contributor

as the name says, and is sometimes substantially faster for large numbers of grids. Has to go with matching amrclaw fasterBound branch. Also has lots of other tiny fixes, mostly format statement enlargements, to also go with larger numbers of grids

@rjleveque
Copy link
Member

The problem is that this was branched off of mjberger/master, which is no longer up to date with master.

I tried merging master in and only had to fix merge conflicts in src/2d/shallow/advanc.f (some rearrangement of timing calls) to get it to merge.

The unit tests then pass on my computer, but won't on travis because this PR depends on clawpack/amrclaw#149, so that should be fixed and merged in first perhaps, or better yet before doing that merge, do a PR on clawpack/clawpack first that includes both these PRs so they get tested together before merging either. See http://www.clawpack.org/developers.html#top-level-pull-requests

@@ -161,10 +162,10 @@ def gulf_after_axes(cd):
plotaxes.ylimits = gulf_ylimits
plotaxes.afteraxes = gulf_after_axes

surge.add_surface_elevation(plotaxes, plot_type='contourf',
surge.plot.add_surface_elevation(plotaxes, plot_type='contourf',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is wrong -- should be surgeplot.add_surface_elevation(...

@mandli: not sure if this is wrong in other branches.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At some point there was a PR that replaced these updates and I have not had a chance to go back through and sync everything up. Let me know if you find any other issues with the data objects as I imagine there are more.

* master: (36 commits)
  bug fixes in maketopo.py of geoclaw/examples/tsunami/chile2010
  add a docstring, not sure if nosetests is running this test.
  added test_etopo.py and sample script fetch_etopo.py
  New etopotools.py module for downloading etopo1 data
  added fgmax_frompatch_arrival_velocity.f90, which can be used instead of the default fgmax_frompatch.f90 to flag arrival times based on amplitude of velocity rather than on surface elevation.  Works better particularly in regions of subsidence
  add abs() to test for equality, and fix more places where delta should be a tuple
  fixed comments about cellsize
  fix delta to tuple in topotools write and plot functions
  topo_type is not second argument, so need to specify name
  added maketopo.py to make bowl.nc in tests/netcdf_topo
  Revert to default compiler flags
  Adapt topotools.Topography.plot to new add_colorbar
  change max to min in topotools
  Add None and False support to parameter options
  Change from rinfinity to huge intrinsic
  Add and modify surge data parameters
  missing .max() operator in computing deltas
  Minor tweak for failed tests
  Fix travis failure script
  Fix compilation bug in NetCDF test"
  ...
* origin/master:
  skiprows replaced by skip_header in call to genfromtxt in dtopotools.py due to numpy changes
  Set omp private variable time in gfixup.f
FasterBound updated to incorporate 5.3.1 changes to master
@rjleveque
Copy link
Member

After consulting with @mandli and @mjberger, decided this is ready to merge.

rjleveque added a commit that referenced this pull request Jan 10, 2016
@rjleveque rjleveque merged commit 2984752 into clawpack:master Jan 10, 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.

None yet

3 participants