search

CERN / TIGRE

TIGRE: Tomographic Iterative GPU-based Reconstruction Toolbox
BSD 3-Clause "New" or "Revised" License
535 stars 180 forks source link

Fix bug and typos #403

Closed twhitbread closed 1 year ago

twhitbread commented 1 year ago

In SART_TV.m, the voxel size used when calculating the W weights is geo.DSD-geo.DSO. This errors when the length of DSD or DSO is greater than 2, i.e. for varying DSD and DSO (e.g. tomosynthesis scans). This PR changes it to match the voxel size used in SART.m. The rest of the changes are typos/cleaning up a bit.

I will not change the voxel sizes used in SIRT or OS-SART yet, as alignment will come when https://github.com/CERN/TIGRE/issues/402 is implemented. This is purely to fix a bug in the short term.

twhitbread commented 1 year ago

Also removed errorL2=[]; from SIRT because it is done twice.

AnderBiguri commented 1 year ago

Thanks @twhitbread ! Too many things in my plate, its taking me longer than expected to fix easy bugs. Thanks for helping!