From 9cecbdfc5697a0f338b30b6ad1e5ff1bdd283254 Mon Sep 17 00:00:00 2001 From: Niels Dekker Date: Mon, 23 Mar 2026 22:17:46 +0100 Subject: [PATCH 1/2] PERF: Call `m_LabelStatusImage->FillBuffer(1)` in MRFImageFilter Replaced a buffered region iteration with the equivalent (but more efficient) `FillBuffer` call, in `MRFImageFilter::Allocate()`. Follow-up to pull request https://github.com/InsightSoftwareConsortium/ITK/pull/5970 "Replace buffered region iteration with FillBuffer in FastMarching filters" --- .../include/itkMRFImageFilter.hxx | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/Modules/Segmentation/MarkovRandomFieldsClassifiers/include/itkMRFImageFilter.hxx b/Modules/Segmentation/MarkovRandomFieldsClassifiers/include/itkMRFImageFilter.hxx index ebf93263c46..d83642e1a2d 100644 --- a/Modules/Segmentation/MarkovRandomFieldsClassifiers/include/itkMRFImageFilter.hxx +++ b/Modules/Segmentation/MarkovRandomFieldsClassifiers/include/itkMRFImageFilter.hxx @@ -346,14 +346,8 @@ MRFImageFilter::Allocate() m_LabelStatusImage->SetBufferedRegion(region); m_LabelStatusImage->Allocate(); - LabelStatusImageIterator rIter(m_LabelStatusImage, m_LabelStatusImage->GetBufferedRegion()); - // Initialize the label status image to 1 - while (!rIter.IsAtEnd()) - { - rIter.Set(1); - ++rIter; - } + m_LabelStatusImage->FillBuffer(1); } template From e0ccde0da3b87d999aec0964eb9da50443fb97d0 Mon Sep 17 00:00:00 2001 From: Niels Dekker Date: Mon, 23 Mar 2026 22:28:09 +0100 Subject: [PATCH 2/2] STYLE: Remove `labelledImageIt` from `RGBGibbsPriorFilter::GibbsEnergy` This particular member function never used its local `labelledImageIt` variable. --- .../include/itkRGBGibbsPriorFilter.hxx | 2 -- 1 file changed, 2 deletions(-) diff --git a/Modules/Segmentation/MarkovRandomFieldsClassifiers/include/itkRGBGibbsPriorFilter.hxx b/Modules/Segmentation/MarkovRandomFieldsClassifiers/include/itkRGBGibbsPriorFilter.hxx index 567ba0eaca7..5506832f592 100644 --- a/Modules/Segmentation/MarkovRandomFieldsClassifiers/include/itkRGBGibbsPriorFilter.hxx +++ b/Modules/Segmentation/MarkovRandomFieldsClassifiers/include/itkRGBGibbsPriorFilter.hxx @@ -288,8 +288,6 @@ template double RGBGibbsPriorFilter::GibbsEnergy(unsigned int i, unsigned int k, unsigned int k1) { - LabelledImageRegionIterator labelledImageIt(m_LabelledImage, m_LabelledImage->GetBufferedRegion()); - LabelType f[8]; unsigned int neighborcount = 0; int simnum = 0;