Skip to content

Conversation

@blowekamp
Copy link
Member

No description provided.

@blowekamp blowekamp changed the title Update itk for development Update itk to 6.0a3 for development May 13, 2025
@blowekamp
Copy link
Member Author

@N-Dekker With updating ITK to 6.0a3 a compilation error is occurring with Elastix. Likely does to the removal of some ITK macros like ITKv5_CONST or something similar. Any recommendations with Elastix and compatibility with ITK 6.0 for development?

@N-Dekker
Copy link
Collaborator

With updating ITK to 6.0a3 a compilation error is occurring with Elastix. Likely does to the removal of some ITK macros like ITKv5_CONST or something similar. Any recommendations with Elastix and compatibility with ITK 6.0 for development?

Thanks for asking, @blowekamp I locally use ITK v6.0a02 when building elastix from the main branch. Do you think anything between ITK v6.0a02 and ITK v6.0a03 triggered the errors?

Which version (commit or tag) of elastix did you try?

@blowekamp
Copy link
Member Author

With updating ITK to 6.0a3 a compilation error is occurring with Elastix. Likely does to the removal of some ITK macros like ITKv5_CONST or something similar. Any recommendations with Elastix and compatibility with ITK 6.0 for development?

Thanks for asking, @blowekamp I locally use ITK v6.0a02 when building elastix from the main branch. Do you think anything between ITK v6.0a02 and ITK v6.0a03 triggered the errors?

Which version (commit or tag) of elastix did you try?

SimpleITK currently is on Elastix "5.2.0". Looking closer at the error is seems to be related to missing trailing ";" after macros. Perhaps a patch to 5.2.0 is in order? I'll look into making a patch.

@blowekamp
Copy link
Member Author

@N-Dekker I made a PR in Elastix with the changes needed: SuperElastix/elastix#1327 It is based on the 5.2.0 tag.

@blowekamp blowekamp force-pushed the update_ITK_for_development branch from 9043617 to 22c1db6 Compare May 14, 2025 20:23
@blowekamp blowekamp force-pushed the update_ITK_for_development branch from 22c1db6 to 8824d81 Compare May 15, 2025 13:24
@blowekamp blowekamp marked this pull request as ready for review May 16, 2025 13:03
@blowekamp blowekamp requested review from dave3d and zivy May 16, 2025 13:04
@blowekamp blowekamp merged commit cd641e9 into SimpleITK:master May 16, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants