diff Framework/Toolbox/DicomStructureSet.cpp @ 1170:1644de437a7b broker

fixes related to swapped normal in sagittal geometry
author Sebastien Jodogne <s.jodogne@gmail.com>
date Wed, 20 Nov 2019 13:09:15 +0100
parents 709aa65aca17
children 3c7cdbf32e2a
line wrap: on
line diff
--- a/Framework/Toolbox/DicomStructureSet.cpp	Wed Nov 20 10:55:44 2019 +0100
+++ b/Framework/Toolbox/DicomStructureSet.cpp	Wed Nov 20 13:09:15 2019 +0100
@@ -373,8 +373,10 @@
     else if (GeometryToolbox::IsParallelOrOpposite
              (isOpposite, slice.GetNormal(), geometry_.GetAxisX()))
     {
-      // plane is constant X
+      // plane is constant X => Sagittal view (remember that in the
+      // sagittal projection, the normal must be swapped)
 
+      
       /*
       Please read the comments in the section above, by taking into account
       the fact that, in this case, the plane has a constant X, not Y (in 
@@ -427,10 +429,6 @@
         slice.ProjectPoint2(x1, y1, p1);
         slice.ProjectPoint2(x2, y2, p2);
 
-        // TODO WHY THIS???
-        y1 = -y1;
-        y2 = -y2;
-
         return true;
       }
     }