Improve inclusion of CL headers. about slicerigsio HOT 5 CLOSED

I don't see any errors on the dashboards, neither Preview nor Stable, on any platforms. We recently changed defaults to enable OpenCL acceleration. If you don't have OpenCL on your computer then you may disable it (it is only used for faster volume reconstruction).

@Sunderlandkyl could you check if we could pass on OpenCL folder(s)?

from slicerigsio.

Thank you for considering @lassoan .

It's not a big issue, I only need SlicerIGSIO as a dependency to build SlicerIGT. Things are not out of scope here.

from slicerigsio.

Yeah, we can add the option to SlicerIGSIO.

SlicerIGSIO_USE_GPU should only be enabled by default if find_package(OpenCL) is successful.
Does find_package(OpenCL) not find the correct paths?

from slicerigsio.

Does find_package(OpenCL) not find the correct paths?

That seems to be the case.

If I specify

-DOpenCL_DIR:PATH=/opt/rocm/opencl

and configure, the result is :

CMake Warning:
  Manually-specified variables were not used by the project:

    OpenCL_DIR

At build :

In file included from /home/arc/src/SlicerIGSIO-SuperBuild9/IGSIO/VolumeReconstruction/vtkIGSIOPasteSliceIntoVolume.cxx:59:
/home/arc/src/SlicerIGSIO-SuperBuild9/IGSIO/VolumeReconstruction/vtkIGSIOPasteSliceIntoVolumeHelperOpenCL.h:61:10: fatal error: 'CL/cl.hpp' file not found
#include <CL/cl.hpp>

from slicerigsio.

Actually, I found an Arch opench-headers package that installs the headers in /usr/include. So this issue becomes moot, closing and sorry for the noise.

from slicerigsio.