Update ITK to f6c37d?

adamrankin · February 22, 2020, 7:45pm

@jcfr @lassoan @pieper (and anyone else you think might want to comment)

SlicerOpenCV is broken with the update to ITKv5, and requires the fix

Could this be either cherry picked or the branch advanced to this commit?

Thanks,
Adam

jcfr · February 22, 2020, 7:57pm

We are currently building Slicer of the official repo:

github.com

Slicer/Slicer/blob/2fcac81c7971c234950d332e9f4d797e8490db3b/SuperBuild/External_ITK.cmake#L33




# Sanity checks
if(DEFINED ITK_DIR AND NOT EXISTS ${ITK_DIR})
  message(FATAL_ERROR "ITK_DIR variable is defined but corresponds to nonexistent directory")
endif()


if(NOT DEFINED ITK_DIR AND NOT Slicer_USE_SYSTEM_${proj})


  ExternalProject_SetIfNotDefined(
    Slicer_${proj}_GIT_REPOSITORY
    "${EP_GIT_PROTOCOL}://github.com/InsightSoftwareConsortium/ITK"
    QUIET
    )


  ExternalProject_SetIfNotDefined(
    Slicer_${proj}_GIT_TAG
    "v5.1rc01"
    QUIET
    )


  set(EXTERNAL_PROJECT_OPTIONAL_CMAKE_CACHE_ARGS)

and currently using this tag v5.1rc01

A new tag just got published, is the fix included ? See https://github.com/InsightSoftwareConsortium/ITK/releases/tag/v5.1rc02

adamrankin · February 23, 2020, 6:30pm

New tag does not have a complete fix. PR submitted to ITK

adamrankin · February 25, 2020, 3:59pm

@jcfr Could we update to a specific commit of ITK, or could I add a patch file for the current tag?

jamesobutler · April 2, 2020, 6:13pm

@adamrankin Slicer updated the ITK version to v5.1rc03 in https://github.com/Slicer/Slicer/commit/0a8f06c62ba1b7b73e54851300370d911801d85a which should help with the SlicerOpenCV issues.

Topic		Replies	Views
ITK5 released, are we ready to start using it? Development itk	2	459	May 31, 2019
Trouble building ITK python wrapping Development build , itk , itk-python	9	656	February 3, 2022
Updating DCMTK to 3.6.5 Development dicom	6	365	February 11, 2020
Build error with itk on ubuntu1804 Development build	2	611	August 23, 2019
Slicer 5.6.1: Summary, Highlights, and Changelog Release Notes	0	676	December 12, 2023

Update ITK to f6c37d?

Related Topics