cholla-hydro / cholla

A GPU-based hydro code
https://github.com/cholla-hydro/cholla/wiki
MIT License
60 stars 32 forks source link

Add Preliminary MHD Integrator #220

Closed bcaddy closed 1 year ago

bcaddy commented 1 year ago

All non-MHD tests pass except those noted in issues #218 and #219 which also don't pass in dev.

This commit primarily adds support for MHD to the Van Leer 3D integrator along with myriad small fixes and modifications. All new features have tests as well.

The MHD implementation still has bugs but needs to be merged in before we reformat the code so that the merging doesn't provoke too many merge conflicts.

MHD

HLLD Riemann Solver

Testing

Reductions

Utilities

Cuda Utilities

DeviceVector

Math Utilities

New namespace and file for math utilities. Currently contains a semi-general rotation and dot product functions

MHD Utilities

Other

bcaddy commented 1 year ago

This is currently a draft and not ready for merging but wanted to make the changes available for code review as the things that are left to do are minor and should be done shortly.

To do

bcaddy commented 1 year ago

Documentation for the runL1Error method has been added to the wiki

bcaddy commented 1 year ago

Converting MHD to using grid_enum is done so I'm converting this from a draft to a full PR.

@alwinm, switching to using the enum was pretty easy. Could you take a look at my additions to the enum and my general usage? I think it's what we talked about but I'd like confirmation and feedback.

bcaddy commented 1 year ago

FYI, I think github actions are having some issues at the moment. Some jobs are getting shutdown signals from somewhere. I double checked and everything builds fine on C-3PO and the containers running on my machine. For now I'm just restarting failed jobs until they pass.

bcaddy commented 1 year ago

I think I figured out the issue with builds randomly failing. The error ended up being too many resources consumed. I reduced the number of jobs that Make launched and it seems to have fixed it