Skip to content

Commit 43d0ad6

Browse files
committed
Fixing tests and adding comment on release notes
1 parent 688187c commit 43d0ad6

File tree

4 files changed

+10
-30
lines changed

4 files changed

+10
-30
lines changed

documentation/release_6.3.htm

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,13 @@ <h3>Changed functionality</h3>
131131

132132
<h3>Bug fixes</h3>
133133
<ul>
134+
<li>
135+
Fixed a bug in the SPECTUB matrix (introduced in 4.0) that inverted the SPECT reconstruction along the axis.
136+
This had resulted with an inconsitent attenuation correction orientation compared to the reonctructed image.
137+
The axis inversion is now reverted, so if your scripts were using an inverted attenuation map they need to be
138+
updated with this release.
139+
See <a href="https://github.com/UCL/STIR/pull/1631"">PR #1631</a>" . </li>
140+
</ul>
134141
<li>
135142
Fixed a bug in the scatter estimation code (introduced in release 5.1.0) if input data is 3D and "cylindrical"
136143
(there was no bug for "blocksoncylindrical" data).

recon_test_pack/SPECT/SPECTUB/README.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ done by the STIR-SPECT team when developing the code. These are in the org direc
55
Sadly, when the files were generated, we made a mistake such that the attenuation map is actually rotated
66
over 90 degrees and flipped in z-direction (as compared to the reconstructed image).
77
This was spotted by Katherine Royston. This means that the "original" reconstructions are actually incorrect.
8-
In addition, the reconstructed images are inverted along x. So the script inverts the reconstructed images along x.
98

109
However, as the test was not designed to check for correctness, but consistency over different STIR versions,
1110
Kris Thielemans decided to keep the files as they are.

recon_test_pack/generate_atten_cylinder_SPECT.par

Lines changed: 0 additions & 24 deletions
This file was deleted.

recon_test_pack/simulate_data_for_tests.sh

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,6 @@ if [ "$generate_images" -eq 1 ]; then
9090
generate_image generate_uniform_cylinder.par
9191
echo "=== make attenuation image"
9292
generate_image generate_atten_cylinder.par
93-
echo "=== make attenuation image for SPECT: to make up for the fliprl of SPECTUB"
94-
generate_image generate_atten_cylinder_SPECT.par
9593
fi
9694

9795
# Function to comment out the specific line in a given file and write to a new file
@@ -110,8 +108,8 @@ uniform_output_file="my_uniform_cylinder_SPECT.hv"
110108
comment_out_line "$uniform_input_file" "$uniform_output_file"
111109

112110
# Paths to the input and output files for attenuation image
113-
atten_input_file="my_atten_image_SPECT.hv"
114-
atten_output_file="my_atten_image_SPECT_modified.hv"
111+
atten_input_file="my_atten_image.hv"
112+
atten_output_file="my_atten_image_SPECT.hv"
115113

116114
# Comment out the specific line in the attenuation image file
117115
comment_out_line "$atten_input_file" "$atten_output_file"
@@ -178,7 +176,7 @@ fi
178176
: ${background_value:=10}
179177
if [ "$SPECT" -eq 1 ]; then
180178
# create SPECT sinograms
181-
./simulate_data.sh "my_uniform_cylinder_SPECT.hv" "my_atten_image_SPECT_modified.hv" "SPECT_test_Interfile_header.hs" "${background_value}" "${suffix}"
179+
./simulate_data.sh "my_uniform_cylinder_SPECT.hv" "my_atten_image_SPECT.hv" "SPECT_test_Interfile_header.hs" "${background_value}" "${suffix}"
182180
if [ $? -ne 0 ]; then
183181
echo "Error running SPECT simulation"
184182
exit 1

0 commit comments

Comments
 (0)