gandalfcode / gandalf

GANDALF (Graphical Astrophysics code for N-body Dynamics And Lagrangian Fluids)
GNU General Public License v2.0
44 stars 12 forks source link

Added back shear waves with isothermal Riemann solver #151

Closed rbooth200 closed 7 years ago

rbooth200 commented 7 years ago

I realized that the HLL Riemann solver does not correctly capture shear waves, so I've now implemented an HLLC like solver that has a constant density as well as pressure and velocity in the mid-state, but allows the transverse velocities to vary. This option is only switched on when the simulation is 'isothermal'. It should work for both the isothermal and locally isothermal EoS.

While the solver is idealistically better, in some simple tests I can't see any difference...

dhubber commented 7 years ago

Why only isothermal? Surely this is just as relevant for adiabatic and other equations of state?

rbooth200 commented 7 years ago

The shear waves were already present with the adiabatic solver, just missing from the isothermal one.