Rearchitect and Reinstate GPU

[onurulgen]

• Reinstated GPU into the project by fixing or implementing missing functions.
• Rearchitected the project to combine GPU and CPU implementations by using virtual functions.
• Increased the performance up to 11 times by using GPU.
• Implemented unit and regression tests for GPU and CPU by using Catch2 framework.
• Developed GitHub Actions for tests, coverage, static code analysis, and producing CPU and GPU executables for Windows, Linux and macOS.

[codecov]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

[github-actions]

✅Code Analysis Results - no issues found! ✅

[msseibel]

I had to add set(CMAKE_CUDA_ARCHITECTURES "native") to the CMakeLists.txt file. Otherwise, I got the following error, "CMAKE_CUDA_ARCHITECTURES must be non-empty if set".

Besides that, the build worked, and I was able to run the program.

[onurulgen]

I had to add set(CMAKE_CUDA_ARCHITECTURES "native") to the CMakeLists.txt file. Otherwise, I got the following error, "CMAKE_CUDA_ARCHITECTURES must be non-empty if set".

Besides that, the build worked, and I was able to run the program.

CMAKE_CUDA_ARCHITECTURES are set in here. If you need to set that, your cmake configure command must've failed while detecting CUDA, right?

[msseibel]

Yes, that is correct. It failed as soon as it reached line 156 enable_language(CUDA).

[Copilot]

Copilot reviewed 670 out of 685 changed files in this pull request and generated 1 comment.

Files not reviewed (15)

• .github/workflows/linux.yml: Language not supported
• .github/workflows/macos.yml: Language not supported
• .github/workflows/windows.yml: Language not supported
• CMakeLists.txt: Language not supported
• Doxyfile.in: Language not supported
• cmake/FindOPENCL.cmake: Language not supported
• cmake/NIFTYREGConfig.cmake.in: Language not supported
• niftyreg_build_version.txt: Language not supported
• reg-apps/CMakeLists.txt: Language not supported
• reg-apps/reg_average.cpp: Language not supported
• reg-apps/reg_benchmark.cpp: Language not supported
• reg-apps/reg_gpuinfo.cpp: Language not supported
• reg-apps/reg_jacobian.cpp: Language not supported
• reg-apps/reg_measure.cpp: Language not supported
• reg-apps/reg_resample.cpp: Language not supported

Comments suppressed due to low confidence (1)

.github/workflows/tests.yml:32

• The use of ${{ matrix.sudo }} might not be necessary for all platforms. Consider removing it or using a conditional statement.

${{ matrix.sudo }} cmake --build build/ --target install --config ${{ matrix.build_type }}