Updated Pangea3 host config with C++17 standard

GEOS-DEV / thirdPartyLibs

Repository to build the GEOSX third party libraries

3 stars 12 forks source link

Updated Pangea3 host config with C++17 standard #253

Closed sframba closed 9 months ago

sframba commented 9 months ago

Companion PR to GEOS' PR #2910, this PR updates the Pangea 3 host config file after upgrade od cuda to version 11.5 on this machine. In particular,

The c++ standard is set to c++17 (as it should be for GEOS)
~~The default hypre implementation on the machine is set to CPU (GPU not supported for all applications)~~ hypre with cuda acceleration seems to be mostly working now
A flag allowing host constexpr call form device code is added, reducing warnings
CUpti profiling via Caliper is enabled by default

victorapm commented 9 months ago

Hi Stefano, can you share which models don't work with hypre GPU?

sframba commented 9 months ago

Hi Stefano, can you share which models don't work with hypre GPU?

Hi @victorapm , this was the way we used to compile everything on Pangea 3 (Power9 CPUs with Nvidia V100+ nvcc 11.0.3), we had some issues with some solvers such as CompositionalMultiphaseFVM . We haven't tested with the latest versions of hypre. Let me put this on hold while I test the latest TPLs on Pangea3

sframba commented 9 months ago

Hi Stefano, can you share which models don't work with hypre GPU?

Hi again @victorapm, it seems that with the latest version of hypre, one of the GEOS unit tests still fails on Pangea3: testExternalSolvers, with only one test failing: Hypre/SolverTestLaplace2D/0.GMRES_ILU. Still, the flow simulation that was holding us back seems to be working now, so I'll re-enable hypre cuda acceleration by default in this PR

victorapm commented 9 months ago

Thanks, Stefano! I will check with @castelletto1 about the failing test