(HydroMethod != 4 || HydroMethod != 6) should be (HydroMethod != 4 && HydroMethod != 6)
(HydroMethod != 4 || HydroMethod != 6)
(HydroMethod != 4 && HydroMethod != 6)
you're right. fixed.
All tests pass. Good job!
@dcollins4096 , could you take a quick look at this one?
Why is this necessary to hard fail if this flag is set? Isn't
completely ignored if neither MHD method is used?
I think the point is that if you set an initial B-field, thinking that you're run an MHD simulation, this will double-check whether you are using an MHD solver.
Fair enough. It's not immediately obvious that not all solvers do all things. Could you make the error message a little more informative, such as
UniformBField requested with a non-MHD solver. Please use one of the MHD solvers.
This error message requires digging through the code to sort out the nature of the incompatibility.
This makes sense to me. Daegene, could you do this? If so, I will merge the PR>
It's been updated with the new error message.
Thanks! Merging now.