Compilation with cpu_CMakeLists.txt fails

[alashworth]

Hi,
there seems to be some missing #ifdef __USE_CPU in filtered_backprojection.cpp. The project does not build from master because several GPU specific functions aren't available.

[build] /home/andrew/workspace/LEAP/src/filtered_backprojection.cpp: In member function ‘bool filteredBackprojection::filterProjections(float*, float*, parameters*, bool)’:
[build] /home/andrew/workspace/LEAP/src/filtered_backprojection.cpp:282:37: error: ‘zeroPadForOffsetScan_GPU’ was not declared in this scope; did you mean ‘zeroPadForOffsetScan’?
[build] 282 | g = zeroPadForOffsetScan_GPU(g, params, g_out);
[build] | ^~~~~~~~~~~~~~~~~~~~~~~~
[build] | zeroPadForOffsetScan
[build] /home/andrew/workspace/LEAP/src/filtered_backprojection.cpp: In member function ‘bool filteredBackprojection::execute(float*, float*, parameters*, bool, bool, bool)’:
[build] /home/andrew/workspace/LEAP/src/filtered_backprojection.cpp:459:41: error: ‘zeroPadForOffsetScan_GPU’ was not declared in this scope; did you mean ‘zeroPadForOffsetScan’?
[build] 459 | g_pad = zeroPadForOffsetScan_GPU(g, params);
[build] | ^~~~~~~~~~~~~~~~~~~~~~~~
[build] | zeroPadForOffsetScan
[build] /home/andrew/workspace/LEAP/src/filtered_backprojection.cpp:505:33: error: ‘cudaFree’ was not declared in this scope
[build] 505 | cudaFree(g_pad);

[kylechampley]

Ya, I'm not surprised. We have kind of neglected the CPU-only version of LEAP for some time. We will have a new release soon and will fix this issue.

[alashworth]

Do you have a timeline for the new release? Alternatively, I could submit a pull request if those are welcome.