Skip to content

Conversation

@hjmjohnson
Copy link
Member

BoneMorphometry/include/itkReplaceFeatureMapNanInfImageFilter.h:81:34:
error: no template named 'ImageRegionConstIterator' in namespace 'itk'

@hjmjohnson hjmjohnson requested a review from thewtex November 10, 2025 19:30
@hjmjohnson hjmjohnson self-assigned this Nov 10, 2025
@hjmjohnson
Copy link
Member Author

@thewtex FYI: The following script is being used to ensure python wrapping of remote modules on mac before attempting on windows.

#!/bin/bash

ITK_REFERENCE_DIR=/Users/johnsonhj/Dashboard/src/ITK
pushd ${ITK_REFERENCE_DIR}
#  pixi shell --environment python
  pixi run build-python
popd

ls /Users/johnsonhj/Dashboard/src/ITK/Modules/Remote/*remote.cmake

for remote_cmake in $(ls ${ITK_REFERENCE_DIR}/Modules/Remote/*remote.cmake); do
  remote_package=$(basename ${remote_cmake} | sed 's/.remote.cmake//g')
  MODULE_COMPLIANCE_LEVEL=$(grep MODULE_COMPLIANCE_LEVEL ${remote_cmake} |sed 's/MODULE_COMPLIANCE_LEVEL *\([^ ]*\)/\1/g') 
  GIT_REPOSITORY=$(grep GIT_REPOSITORY ${remote_cmake} |sed 's/GIT_REPOSITORY *\([^ ]*\)/\1/g') 
  GIT_PRIMARY_BRANCH=$(git symbolic-ref --short refs/remotes/origin/HEAD 2>/dev/null | sed 's|^origin/||')
  GIT_TAG=$(grep GIT_TAG ${remote_cmake} |sed 's/GIT_TAG *\([^ ]*\)/\1/g') 
  echo "============================ ${remote_package} ============"
  echo "Using: ${GIT_REPOSITORY} at ${GIT_TAG}"
  sleep 3
  if [ ! -d ${remote_package} ]; then
     git clone ${GIT_REPOSITORY} ${remote_package}
  fi 
  pushd ${remote_package}
  git fetch
  # git checkout ${GIT_TAG}
  git checkout ${GIT_GIT_PRIMARY_BRANCH}
  cmake -S . -B cmake-build-release -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=ON -DBUILD_TESTING=ON -DITK_DIR=/Users/johnsonhj/Dashboard/src/ITK/build-python
  ninja -C cmake-build-release
  popd
done

@hjmjohnson hjmjohnson closed this Nov 11, 2025
@hjmjohnson hjmjohnson force-pushed the fix-missing-include-header branch from 71edfdf to 4dc0acd Compare November 11, 2025 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants