Added variable coefficient traffic Rimann solver.

[BrisaDavis]

No description provided.

[mandli]

Can you add the changes needed to make this callable by PyClaw as well? There are some instructions in the Readme.md at the top level of this repository.

[mandli]

I think the build is failing of a naming mistake somewhere. The build is looking for rp1_traffic_vc.f90 rather than rp1_trafficvc.f90.

[rjleveque]

I think rp1_traffic_vc.f90 would be preferable, with the underscore.

I notice we're not very consistent in naming convention, e.g. riemann/src contains
rp1_acoustics_variable.f90
rpn2_vc_acoustics.f90

If we were doing it over I'd prefer rpn2_acoustics_vc.f90 so it would show up with rpn2_acoustics.f90 but not worth redoing things.

[mandli]

@rjleveque that makes sense to me, if we are going to do this we should do it before the release.

[ketch]

I agree with the renaming. Any others that ought to be renamed, while we're at it?

[mandli]

Maybe we should have the basic equations first and any specific version second such as these:

• rp1_layered_shallow_water.f90 to rp1_shallow_water_layered.f90
• rp1_nonlinear_elasticity_fwave.f90 to rp1_elasticity_nonlinear_fwave.f90
• rp1_reactive_euler_with_efix.f90 to rp1_euler_reactive_with_efix.f90

[rjleveque]

I like Kyle's suggestion.

[ketch]

Me too. Since it's a bit confusing to discuss here, I've opened a new issue:

#50