diff --git a/docs/source/changelog.rst b/docs/source/changelog.rst index a0fde51ed..2f2e28821 100644 --- a/docs/source/changelog.rst +++ b/docs/source/changelog.rst @@ -14,6 +14,13 @@ Winston-Lutz image inversion manually between the class instantiation and the ``.analyze()`` call, this change should not affect you. If you are, you may no longer need the inversion call. +ACR +^^^ + +* The slice thickness given when calling ``ACRMRI.results()`` was reporting the nominal slice thickness + not the measured slice thickness. The output from ``ACRMRI.results_data()`` was correct however and has not changed. + + v 3.19.0 -------- diff --git a/pylinac/acr.py b/pylinac/acr.py index 10f18bc54..d0182a30a 100644 --- a/pylinac/acr.py +++ b/pylinac/acr.py @@ -1246,7 +1246,7 @@ def results(self, as_str: bool = True) -> str | tuple: string = ( f" - {self._model} Results - ", f"Geometric Distortions: {self.geometric_distortion.distances()}", - f"Slice Thickness: {self.slice1.slice_thickness:2.2f}mm", + f"Slice Thickness: {self.slice1.measured_slice_thickness_mm:2.2f}mm", f"Slice 1 S/I Position shift: {self.slice1.slice_shift_mm:2.2f}mm", f"Slice 11 S/I Position shift: {self.slice11.slice_shift_mm:2.2f}mm", f"Uniformity PIU: {self.uniformity_module.percent_image_uniformity:2.2f}", diff --git a/tests_basic/test_acr.py b/tests_basic/test_acr.py index 98e0ed4ec..b15e5a0a4 100644 --- a/tests_basic/test_acr.py +++ b/tests_basic/test_acr.py @@ -316,6 +316,7 @@ class ACRMRMixin(CloudFileMixin): slice1_shift: float slice11_shift: float psg: float + results: list[str] = [] @classmethod def setUpClass(cls): @@ -356,6 +357,11 @@ def test_slice11_shift(self): def test_psg(self): self.assertAlmostEqual(self.mri.uniformity_module.psg, self.psg, delta=0.3) + def test_results(self): + results = self.mri.results() + for result in self.results: + self.assertIn(result, results) + class ACRT1Single(ACRMRMixin, TestCase): file_name = "T1-Single.zip" @@ -413,6 +419,7 @@ class ACRGE3T(ACRMRMixin, TestCase): slice1_shift = 0 slice11_shift = 1.5 psg = 0.3 + results = ["5.50mm"] class ACRGE3TOffset(ACRGE3T): @@ -439,6 +446,7 @@ class ACRGE3TRotated(ACRGE3T): """ phantom_roll = -0.4 + results = ["4.80mm"] # induced rotation does change this a bit. See above. @classmethod def setUpClass(cls):