search

AMReX-Fluids / AMReX-Hydro

AMReX-based hydro routines
https://amrex-fluids.github.io/amrex-hydro/docs_html
Other
11 stars 24 forks source link

Add missing RZ pieces in HydroUtils #89

Closed esclapez closed 2 years ago

cgilet commented 2 years ago

Could we combine the Cartesian and RZ versions of ComputeFluxes and ComputeDivergence? I can't think of a good reason to maintain separate versions, especially if we're passing geom.

esclapez commented 2 years ago

I need to test this on GPU even though 2D+GPU is rarely a priority.

esclapez commented 2 years ago

@cgilet successfully tested on GPU.