I believe that with this PR all the bugs in MDH are squashed. There are still some fully 3D tests to run but I've run five different MHD shock tubes in X, Y, Z, and both left and right orientations so I doubt there are any significant bugs remaining. I've attached some plots of those shock tubes and you can find the fiducial plots to compare them against in the MHD Project within the card for that shock tube.
Make the state selection more explicit in the HLLD solver. Before I was using a set of single speed checks to verify the correct state was chosen. Now I'm using the full 1-2 speed checks; if this becomes a performance issue we can figure out out to reduce it to single speed checks at that time.
The double star state check for zero magnetic field was comparing against Bx not Bx^2 which led to issues with negative Bx.
Fix initial conditions for the Dai & Woodward and Ryu & Jones 2a shock tubes. There were typos in the papers I was referencing. These initial conditions now agree with the plots in those papers and the actual values used in Athena++.
New Tests
Additional HLLD test case to highlight bug 2 above if it ever reappears
Slightly refactor some MHD system tests so that they can share the same test suite and not duplicate as much code
Shock tube tests for the following shock tubes
Brio & Wu
Dai & Woodward
Ryu & Jones 1a
Ryu & Jones 2a
Ryu & Jones 4d
Other
Add bin/* to .gitignore to avoid accidentally committing stuff in bin
General
I believe that with this PR all the bugs in MDH are squashed. There are still some fully 3D tests to run but I've run five different MHD shock tubes in X, Y, Z, and both left and right orientations so I doubt there are any significant bugs remaining. I've attached some plots of those shock tubes and you can find the fiducial plots to compare them against in the MHD Project within the card for that shock tube.
MHD Shock Tubes.zip
Bug Fixes
Bx
notBx^2
which led to issues with negativeBx
.New Tests
Other
bin/*
to.gitignore
to avoid accidentally committing stuff inbin