riemann icon indicating copy to clipboard operation
riemann copied to clipboard

Published 20 hours ago • verified publisher icon clawpack

Adding Roe solver entropy fix and an exact Riemann solver

Open orevans opened this issue 9 years ago • 2 comments

orevans avatar Dec 16 '15 16:12 orevans

Other than the comment above it would be nice to add a test for both Riemann solvers in PyClaw.

mandli avatar Dec 17 '15 14:12 mandli

This fails on Travis because we don't have scipy installed there for the Riemann repo. We could add it, or simply move the import inside the roe solver function. I prefer the latter; @orevans could you please move that import?

ketch avatar Jan 28 '16 11:01 ketch