search

geodynamics / Rayleigh

Rayleigh: Pseudo-spectral MHD
GNU General Public License v3.0
59 stars 49 forks source link

Add j2001 hydro test #559

Open gassmoeller opened 1 month ago

gassmoeller commented 1 month ago

Fix #557 by adding the j2011 test cases to the test suite. Let's see how the testers react.

feathern commented 1 month ago

I would suggest removing the MHD test for now while we work on building something with a non-random initial condition. I just ran the hydro test on my machine and it's good. Also, I did a quick sanity check to make sure that significant changes in the measured variables were occurring after such a short number of timesteps, and they are, so I think the hydro test is great. I can approve and merge asap once the MHD test is removed (assuming you haven't already come up with a fix).