You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
Configuring and compiling geos with the TOTAL/pangea3-gcc8.4.1-openmpi-4.1.2.cmake host-config file and the -DENABLE_CUDA=OFF option fails.
As the cuda compilation is very slow, it is sometimes useful to be able to be able to quickly disable it without changing anything else in your configuration.
To Reproduce
Steps to reproduce the behavior:
Log on pangea 3
load the modules needed to build the TPls and GEOS ( 1) gcc/8.4.1 2) cuda/11.5.0 3) ompi/4.1.2 4) openblas/0.3.18 5) cmake/3.26.4)
Configure and build the TPLs with TOTAL/pangea3-gcc8.4.1-openmpi-4.1.2.cmake host-config file and the -DENABLE_CUDA=OFF option
Configure and build Geos with the same arguments (the TOTAL/pangea3-gcc8.4.1-openmpi-4.1.2.cmake host-config file and the -DENABLE_CUDA=OFF option)
The Geos build fails due :
to the HYPRE solver setted to "CUDA" in the host-config file of the TPLs: the ENABLE_CUDA variable sould be tested and the setting of HYPRE to CUDA should be done only if ENABLE_CUDA is ON
to the CUDA::nvToolsExt added inconsistently in CMake (ENABLE_CUDA flag sometimes tested and sometimes not). As this module cannot work if ENABLE_CUDA is OFF, it makes sense (and would be easier for the user) to test the ENABLE_CUDA variable everywhere.
Expected behavior
As a user, I have tried to configure and build the project without looking at the host-config file and the exposed ENABLE_CUDA variable make me think that I could disable cuda with this single command line argument, which would be very convenient.
In fact it fails because:
dependencies to cuda are added by the host-config file of the TPLs without checking if cuda has been explicitely disabled
the nvtools cannot work without cuda but for now, the ENABLE_CUDA_NVTOOLSEXT has to be setted explicitely to OFF too.
I think that the cmake of the project should avoid failing combination of options. I will try to propose PRs accordingly.
Platform (please complete the following information):
Describe the bug
Configuring and compiling geos with the
TOTAL/pangea3-gcc8.4.1-openmpi-4.1.2.cmake
host-config file and the-DENABLE_CUDA=OFF
option fails.As the cuda compilation is very slow, it is sometimes useful to be able to be able to quickly disable it without changing anything else in your configuration.
To Reproduce
Steps to reproduce the behavior:
1) gcc/8.4.1 2) cuda/11.5.0 3) ompi/4.1.2 4) openblas/0.3.18 5) cmake/3.26.4
)TOTAL/pangea3-gcc8.4.1-openmpi-4.1.2.cmake
host-config file and the-DENABLE_CUDA=OFF
optionTOTAL/pangea3-gcc8.4.1-openmpi-4.1.2.cmake
host-config file and the-DENABLE_CUDA=OFF
option)ENABLE_CUDA
variable sould be tested and the setting of HYPRE to CUDA should be done only ifENABLE_CUDA
isON
CUDA::nvToolsExt
added inconsistently in CMake (ENABLE_CUDA
flag sometimes tested and sometimes not). As this module cannot work ifENABLE_CUDA
is OFF, it makes sense (and would be easier for the user) to test the ENABLE_CUDA variable everywhere.Expected behavior
As a user, I have tried to configure and build the project without looking at the host-config file and the exposed
ENABLE_CUDA
variable make me think that I could disable cuda with this single command line argument, which would be very convenient.In fact it fails because:
ENABLE_CUDA_NVTOOLSEXT
has to be setted explicitely to OFF too.I think that the cmake of the project should avoid failing combination of options. I will try to propose PRs accordingly.
Platform (please complete the following information):
Additional context
-DENABLE_CUDA=OFF
cmake optionLinked to : #3129, GEOS-DEV/thirdPartyLibs#264
The text was updated successfully, but these errors were encountered: