Randall J. LeVeque

Has been done in 2d in PR #36. Not yet in 3d.

@cjvogl: So I think a better way to do this is to write `allowflag` to only allow flagging a cell if it is more than `ibuff` cells away from the...

You are right that `gamma1` is not needed, although the code should still run fine either way since `gamma1` is defined locally, and in Fortran a common block is just...

Yes, best to use lapack if available. But we could provide something like we do for this example: https://github.com/clawpack/apps/blob/master/fvmbook/chap17/advdiff/tridiag.f

Or should we just cleanly skip this unit test if lapack isn't available, as happens with the netCDF tests?

I also agree it would be best to make the tests more verbose and easier to understand and modify. I wonder about running convergence tests, since part of the point...

Thanks @ketch and @carlosmunozmoncayo , you've done a lot of work on this! Many of the tests pass for me, but the one in `examples/shallow_2d/radial_dam_break.py` fails with: ``` E AssertionError:...

Thanks @BarrySmith! @mjberger tells me we still need to do some more timing comparisons between this and the original version, which I will try to work with her on in...

I tried running this along with my new petscMPIoptions file from #631. It still runs fine with ``` -mpi_linear_solver_server_use_shared_memory false ``` but with ``` -mpi_linear_solver_server_use_shared_memory true ``` I get the...

I just tried this on a different problem, and again it runs ok with `..use_shared_memory false` but when `true` I now get a different error, see below. This problem is...