markus-jehl

Perfect, thanks! It builds now, but importing stir in python leads to an error still: `/workspace/python-reconstruction-pipeline/libs/install/stir/python/_stir.so: undefined symbol: _ZN4stir5ArrayILi1EfEC1ERKNS_10IndexRangeILi1EEEPf` I'll go through the code changes in patience to understand all...

Ran the latest version this morning through the python reconstruction steps and everything worked fine! Will later also test it in C++, as well as with sanitisers

Not noticeably in the setup I used, but I want to test this with C++ where everything is a bit more controllable.

Tested it now in C++ and both speed and memory consumption look identical.

Once this is fixed, the tolerance in `check_symmetry` function in `test_interpolate_scatter.cxx` can be tightened.

I agree, it looks like the second loop should be be looping over the blocks per bucket. I wonder why I never ran into any issues with this - will...

Yep, that's it - I never use more than one bucket axially. Thanks!

sure, I'll put the second one on my list

Should this be closed and a new one created that refers specifically to investigating methods to replicate tangential lines of response, @KrisThielemans ?

IMPORTANT: currently, parallelproj truncates the resulting images to a cylinder with the inner ring radius of the scanner. This means any activity in the corners of a BlocksOnCylindrical scanner will...