WIP: PERF: Performance acceleration via GPU-accelerated FFT's #1331
Conversation
To address: /home/matt/src/ANTs/Examples/PrintHeader.cxx: In function ‘int ants::PrintHeader(int, char**)’: /home/matt/src/ANTs/Examples/PrintHeader.cxx:284:34: error: ‘ITK_COORDINATE_ORIENTATION_INVALID’ is not a member of ‘itk::SpatialOrientation’; did you mean ‘ITK_COORDINATE_ORIENTATION_IAL’? 284 | itk::SpatialOrientation::ITK_COORDINATE_ORIENTATION_INVALID;
|
|
||
| # Set up ITK | ||
| find_package(ITK ${ITK_VERSION_ID} REQUIRED) | ||
| find_package(ITK ${ITK_VERSION_ID} REQUIRED |
There was a problem hiding this comment.
This block breaks the default SuperBuild for me. It's asking for ITK components that aren't built by default:
VkFFTBackend
ITKFEM
ITKVTK
ITKVtkGlue
I don't know about ITKFEM, but the others should be required only if the relevant build options are turned on.
There was a problem hiding this comment.
ITKFEM is because Temporary/ has includes which use it.
VkFFTBackend is the code which supports this new feature.
ITKVTK and ITKVTK need to be wrapped so they only are required if the USE_VTK is on.
I'm working on this
There was a problem hiding this comment.
also Examples/ConformalMapping.cxx and Examples/ANTSConformalMapping.cxx but it appears these are not built, since ITKFEM module hasn't been enabled and the build still works.
There was a problem hiding this comment.
I was working on this as well, but I could not build with vkFFT support. It stops with this error
make
[ 47%] Built target ITKv5
[ 52%] Performing forcebuild step for 'ANTS'
[ 58%] Performing build step for 'ANTS'
[ 1%] Building CXX object Examples/CMakeFiles/antsUtilities.dir/antsRegistrationTemplateHeader.cxx.o
In file included from /Users/pcook/tmp/NOT_BACKED_UP/antsGPU/ANTs/Examples/antsRegistrationTemplateHeader.cxx:1:
In file included from /Users/pcook/tmp/NOT_BACKED_UP/antsGPU/ANTs/Examples/antsRegistrationTemplateHeader.h:14:
In file included from /Users/pcook/tmp/NOT_BACKED_UP/antsGPU/ANTs/Examples/itkantsRegistrationHelper.h:1472:
In file included from /Users/pcook/tmp/NOT_BACKED_UP/antsGPU/ANTs/Examples/itkantsRegistrationHelper.hxx:7:
In file included from /Users/pcook/tmp/NOT_BACKED_UP/antsGPU/build/staging/include/ITK-5.3/itkVkHalfHermitianToRealInverseFFTImageFilter.h:24:
/Users/pcook/tmp/NOT_BACKED_UP/antsGPU/build/staging/include/ITK-5.3/itkVkCommon.h:24:10: fatal error: 'vkFFT.h' file not found
#include "vkFFT.h"
^~~~~~~~~
1 error generated.
make[5]: *** [Examples/CMakeFiles/antsUtilities.dir/antsRegistrationTemplateHeader.cxx.o] Error 1
make[4]: *** [Examples/CMakeFiles/antsUtilities.dir/all] Error 2
make[3]: *** [all] Error 2
make[2]: *** [ANTS-prefix/src/ANTS-stamp/ANTS-build] Error 2
make[1]: *** [CMakeFiles/ANTS.dir/all] Error 2
make: *** [all] Error 2
Have you been able to compile with the vtFFT module on?
|
Oh, I guess edits aren't turned on, so my changes went to a local branch. https://github.com/ANTsX/ANTs/tree/perf-fft Anyhow, I'm not sure if this PR is even still interesting, I was just curious to see if I could make it build. I got as far as it complaining about lack of Vulkan on my Mac. |
3 participants
Further discussion today at the ANTs Get Your Brain Straight BoF:
https://insightsoftwareconsortium.github.io/GetYourBrainStraight/HCK01_2022_Virtual/