STYLE: Do not set GridSize when TransformIsStackTransform, in Metrics

The GridSize is only relevant when the transform is _not_ a stack transform!
SuperElastix · Jul 1, 2024 · 7259081 · 7259081
1 parent fd2e462
commit 7259081
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 36 deletions.
diff --git a/Components/Metrics/PCAMetric/elxPCAMetric.hxx b/Components/Metrics/PCAMetric/elxPCAMetric.hxx
@@ -115,15 +115,6 @@ PCAMetric<TElastix>::BeforeEachResolution()
         /** Set itk member variable. */
         this->SetTransformIsStackTransform(true);
 
-        if (const unsigned int numberOfSubTransforms{ stackTransform->GetNumberOfSubTransforms() };
-            numberOfSubTransforms > 0)
-        {
-          /** Check if subtransform is a B-spline transform. */
-          if (dynamic_cast<ReducedDimensionBSplineTransformBaseType *>(stackTransform->GetSubTransform(0).GetPointer()))
-          {
-            this->SetGridSize(FixedImageSizeType::Filled(numberOfSubTransforms));
-          }
-        }
         // Return early, now that the current transform is a stack transform.
         return;
       }

diff --git a/Components/Metrics/PCAMetric2/elxPCAMetric2.hxx b/Components/Metrics/PCAMetric2/elxPCAMetric2.hxx
@@ -108,15 +108,6 @@ PCAMetric2<TElastix>::BeforeEachResolution()
         /** Set itk member variable. */
         this->SetTransformIsStackTransform(true);
 
-        if (const unsigned int numberOfSubTransforms{ stackTransform->GetNumberOfSubTransforms() };
-            numberOfSubTransforms > 0)
-        {
-          /** Check if subtransform is a B-spline transform. */
-          if (dynamic_cast<ReducedDimensionBSplineTransformBaseType *>(stackTransform->GetSubTransform(0).GetPointer()))
-          {
-            this->SetGridSize(FixedImageSizeType::Filled(numberOfSubTransforms));
-          }
-        }
         // Return early, now that the current transform is a stack transform.
         return;
       }

diff --git a/...Metrics/SumOfPairwiseCorrelationsMetric/elxSumOfPairwiseCorrelationCoefficientsMetric.hxx b/...Metrics/SumOfPairwiseCorrelationsMetric/elxSumOfPairwiseCorrelationCoefficientsMetric.hxx
@@ -109,15 +109,6 @@ SumOfPairwiseCorrelationCoefficientsMetric<TElastix>::BeforeEachResolution()
         /** Set itk member variable. */
         this->SetTransformIsStackTransform(true);
 
-        if (const unsigned int numberOfSubTransforms{ stackTransform->GetNumberOfSubTransforms() };
-            numberOfSubTransforms > 0)
-        {
-          /** Check if subtransform is a B-spline transform. */
-          if (dynamic_cast<ReducedDimensionBSplineTransformBaseType *>(stackTransform->GetSubTransform(0).GetPointer()))
-          {
-            this->SetGridSize(FixedImageSizeType::Filled(numberOfSubTransforms));
-          }
-        }
         // Return early, now that the current transform is a stack transform.
         return;
       }

diff --git a/Components/Metrics/VarianceOverLastDimension/elxVarianceOverLastDimensionMetric.hxx b/Components/Metrics/VarianceOverLastDimension/elxVarianceOverLastDimensionMetric.hxx
@@ -138,15 +138,6 @@ VarianceOverLastDimensionMetric<TElastix>::BeforeEachResolution()
         /** Set itk member variable. */
         this->SetTransformIsStackTransform(true);
 
-        if (const unsigned int numberOfSubTransforms{ stackTransform->GetNumberOfSubTransforms() };
-            numberOfSubTransforms > 0)
-        {
-          /** Check if subtransform is a B-spline transform. */
-          if (dynamic_cast<ReducedDimensionBSplineTransformBaseType *>(stackTransform->GetSubTransform(0).GetPointer()))
-          {
-            this->SetGridSize(FixedImageSizeType::Filled(numberOfSubTransforms));
-          }
-        }
         // Return early, now that the current transform is a stack transform.
         return;
       }