COMP: Update CI to ITKRemoteModuleBuildTestPackageAction v5.4.6 and Python 3.10+

hjmjohnson · claude · hjmjohnson · commit 2b660319cc74 · 2026-04-03T18:04:59.000-05:00
- Update clang-format linter from @master to @main with checkout@v5 - Bump reusable workflow from v5.4.0 to v5.4.6 (fixes macos-13 deprecation, adds GHCR dockcross pre-pull with retry) - Update minimum Python version to 3.10+ - Apply clang-format to match ITK coding style Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
diff --git a/.github/workflows/build-test-package.yml b/.github/workflows/build-test-package.yml
@@ -12,9 +12,9 @@ on:
 
 jobs:
   cxx-build-workflow:
-    uses: InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction/.github/workflows/build-test-cxx.yml@v5.4.0
+    uses: InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction/.github/workflows/build-test-cxx.yml@v5.4.6
 
   python-build-workflow:
-    uses: InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction/.github/workflows/build-test-package-python.yml@v5.4.0
+    uses: InsightSoftwareConsortium/ITKRemoteModuleBuildTestPackageAction/.github/workflows/build-test-package-python.yml@v5.4.6
     secrets:
       pypi_password: ${{ secrets.pypi_password }}
diff --git a/.github/workflows/clang-format-linter.yml b/.github/workflows/clang-format-linter.yml
@@ -7,6 +7,8 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v5
 
-    - uses: InsightSoftwareConsortium/ITKClangFormatLinterAction@master
+    - uses: InsightSoftwareConsortium/ITKClangFormatLinterAction@main
+      with:
+        itk-branch: main
diff --git a/include/itkVariationalRegistrationElasticRegularizer.h b/include/itkVariationalRegistrationElasticRegularizer.h
@@ -189,9 +189,9 @@ class VariationalRegistrationElasticRegularizer : public VariationalRegistration
   typename FFTWProxyType::PlanType m_PlanForward[ImageDimension];  /** FFT forward plan  */
   typename FFTWProxyType::PlanType m_PlanBackward[ImageDimension]; /** FFT backward plan */
   typename FFTWProxyType::ComplexType *
-                                      m_ComplexBuffer[ImageDimension]; /** memory space for output of forward and input of backward FFT*/
-  typename FFTWProxyType::PixelType * m_InputBuffer;                   /** FFT memory space for input data */
-  typename FFTWProxyType::PixelType * m_OutputBuffer;                  /** FFT memory space for output data */
+    m_ComplexBuffer[ImageDimension];                  /** memory space for output of forward and input of backward FFT*/
+  typename FFTWProxyType::PixelType * m_InputBuffer;  /** FFT memory space for input data */
+  typename FFTWProxyType::PixelType * m_OutputBuffer; /** FFT memory space for output data */
 
   struct ElasticFFTThreadStruct
   {
diff --git a/include/itkVariationalRegistrationFunction.h b/include/itkVariationalRegistrationFunction.h
@@ -19,7 +19,7 @@
 #define itkVariationalRegistrationFunction_h
 
 #include "itkFiniteDifferenceFunction.h"
-//#include "itkWarpImageFilter.h"
+// #include "itkWarpImageFilter.h"
 #include "itkContinuousBorderWarpImageFilter.h"
 #include <mutex>
 
diff --git a/include/itkVariationalRegistrationRegularizer.h b/include/itkVariationalRegistrationRegularizer.h
@@ -90,7 +90,7 @@ class VariationalRegistrationRegularizer : public InPlaceImageFilter<TDisplaceme
 
   /** Initialize the filter. */
   virtual void
-  Initialize(){};
+  Initialize() {};
 
 private:
   /** A boolean that indicates, if image spacing is considered. */
diff --git a/pyproject.toml b/pyproject.toml
@@ -34,7 +34,7 @@ classifiers = [
     "Topic :: Scientific/Engineering :: Medical Science Apps.",
     "Topic :: Software Development :: Libraries",
 ]
-requires-python = ">=3.8"
+requires-python = ">=3.10"
 dependencies = [
     "itk",
 ]
diff --git a/src/VariationalRegistrationMain.cxx b/src/VariationalRegistrationMain.cxx
@@ -640,8 +640,7 @@ main(int argc, char * argv[])
   FunctionType::Pointer function;
   switch (forceType)
   {
-    case 0:
-    {
+    case 0: {
       DemonsFunctionType::Pointer demonsFunction = DemonsFunctionType::New();
       switch (forceDomain)
       {
@@ -659,8 +658,7 @@ main(int argc, char * argv[])
       function = demonsFunction;
     }
     break;
-    case 1:
-    {
+    case 1: {
       SSDFunctionType::Pointer ssdFunction = SSDFunctionType::New();
       switch (forceDomain)
       {
@@ -678,8 +676,7 @@ main(int argc, char * argv[])
       function = ssdFunction;
     }
     break;
-    case 2:
-    {
+    case 2: {
       NCCFunctionType::Pointer    nccFunction = NCCFunctionType::New();
       NCCFunctionType::RadiusType r;
       for (unsigned int dim = 0; dim < NCCFunctionType::ImageDimension; dim++)
@@ -722,22 +719,19 @@ main(int argc, char * argv[])
   RegularizerType::Pointer regularizer;
   switch (regularizerType)
   {
-    case 0:
-    {
+    case 0: {
       GaussianRegularizerType::Pointer gaussRegularizer = GaussianRegularizerType::New();
       gaussRegularizer->SetStandardDeviations(std::sqrt(regulVar));
       regularizer = gaussRegularizer;
     }
     break;
-    case 1:
-    {
+    case 1: {
       DiffusionRegularizerType::Pointer diffRegularizer = DiffusionRegularizerType::New();
       diffRegularizer->SetAlpha(regulAlpha);
       regularizer = diffRegularizer;
     }
     break;
-    case 2:
-    {
+    case 2: {
 #if defined(ITK_USE_FFTWD) || defined(ITK_USE_FFTWF)
       ElasticRegularizerType::Pointer elasticRegularizer = ElasticRegularizerType::New();
       elasticRegularizer->SetMu(regulMu);
@@ -748,8 +742,7 @@ main(int argc, char * argv[])
 #endif
     }
     break;
-    case 3:
-    {
+    case 3: {
 #if defined(ITK_USE_FFTWD) || defined(ITK_USE_FFTWF)
       CurvatureRegularizerType::Pointer curvatureRegularizer = CurvatureRegularizerType::New();
       curvatureRegularizer->SetAlpha(regulAlpha);
@@ -775,20 +768,17 @@ main(int argc, char * argv[])
   RegistrationFilterType::Pointer regFilter;
   switch (searchSpace)
   {
-    case 0:
-    {
+    case 0: {
       regFilter = RegistrationFilterType::New();
       break;
     }
-    case 1:
-    {
+    case 1: {
       DiffeomorphicRegistrationFilterType::Pointer diffeoRegFilter = DiffeomorphicRegistrationFilterType::New();
       diffeoRegFilter->SetNumberOfExponentiatorIterations(numberOfExponentiatorIterations);
       regFilter = diffeoRegFilter;
       break;
     }
-    case 2:
-    {
+    case 2: {
       SymmetricDiffeomorphicRegistrationFilterType::Pointer symmDiffeoRegFilter =
         SymmetricDiffeomorphicRegistrationFilterType::New();
       symmDiffeoRegFilter->SetNumberOfExponentiatorIterations(numberOfExponentiatorIterations);
diff --git a/src/itkVariationalRegistrationIncludeRequiredIOFactories.h b/src/itkVariationalRegistrationIncludeRequiredIOFactories.h
@@ -28,7 +28,7 @@
 #include "itkNrrdImageIOFactory.h"
 #include "itkGiplImageIOFactory.h"
 #include "itkNiftiImageIOFactory.h"
-//#include "itkTestDriverInclude.h"
+// #include "itkTestDriverInclude.h"
 #include "itkObjectFactoryBase.h"
 
 void
diff --git a/src/win32_compatibility/getopt.c b/src/win32_compatibility/getopt.c
@@ -59,7 +59,9 @@ char * optarg;  /* argument associated with option */
  * getopt --
  *	Parse argc/argv argument vector.
  */
-int            getopt(nargc, nargv, ostr) int nargc;
+int
+               getopt(nargc, nargv, ostr)
+int            nargc;
 char * const * nargv;
 const char *   ostr;
 {

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ classifiers = [`
`34`	`34`	`"Topic :: Scientific/Engineering :: Medical Science Apps.",`
`35`	`35`	`"Topic :: Software Development :: Libraries",`
`36`	`36`	`]`
`37`		`-requires-python = ">=3.8"`
	`37`	`+requires-python = ">=3.10"`
`38`	`38`	`dependencies = [`
`39`	`39`	`"itk",`
`40`	`40`	`]`