Skip to content

COMP: Advance remote-module pins to upstream default-branch HEAD#6687

Open
hjmjohnson wants to merge 1 commit into
InsightSoftwareConsortium:mainfrom
hjmjohnson:main-remote-pins
Open

COMP: Advance remote-module pins to upstream default-branch HEAD#6687
hjmjohnson wants to merge 1 commit into
InsightSoftwareConsortium:mainfrom
hjmjohnson:main-remote-pins

Conversation

@hjmjohnson

Copy link
Copy Markdown
Member

Advance 11 remote-module GIT_TAG pins to their upstream default-branch HEAD to reduce drift and pick up maintenance since the last bump.

Local verification: a clean Release build of the 8 buildable updated modules completed with zero build failures on macOS/arm64 (Apple clang). CudaCommon, IOOpenSlide, and Ultrasound are advanced but only exercisable in CI (no CUDA / OpenSlide / compatible VTK locally).

Per-module disposition

Built green locally: BioCell, HASI, PerformanceBenchmarking, SCIFIO, Shape, SkullStrip, VkFFTBackend, WebAssemblyInterface.

Advanced, CI-verified only: CudaCommon, IOOpenSlide, Ultrasound.

Left at existing pins (not advanced):

Update remote-module GIT_TAG pins to their upstream default-branch HEAD
to reduce drift and pick up maintenance since the last bump.

RTK, SphinxExamples, TubeTK, Cleaver, and TractographyTRX are left at
their existing pins: RTK and TractographyTRX await upstream fixes
(RTKConsortium/RTK#975, tee-ar-ex/ITKTractographyTRX#30), TubeTK has an
upstream header/implementation mismatch, SphinxExamples is only
verifiable via its standalone build, and Cleaver's bundled GoogleTest
collides with ITK's when BUILD_TESTING is on.

Verified by a clean Release build of the 8 buildable updated modules
(zero build failures). CudaCommon, IOOpenSlide, and Ultrasound are
advanced but only exercisable in CI (no CUDA / OpenSlide / a compatible
VTK on the local macOS host).
@github-actions github-actions Bot added type:Compiler Compiler support or related warnings area:Remotes Issues affecting the Remote module labels Jul 23, 2026
@hjmjohnson
hjmjohnson marked this pull request as ready for review July 23, 2026 02:43
@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR advances 11 optional remote modules to newer upstream commits. The main changes are:

  • Eight module pins validated by a local macOS Release build.
  • Three dependency-specific module pins left for CI coverage.
  • No changes to module repositories, compliance levels, or fetch behavior.

Confidence Score: 5/5

This looks safe to merge after the dependency-specific CI jobs pass.

The diff only replaces immutable remote-module commit pins. No concrete build, runtime, API, or Eigen-include failure was established.

CUDA, OpenSlide, and VTK paths still depend on their CI environments: Modules/Remote/CudaCommon.remote.cmake, Modules/Remote/IOOpenSlide.remote.cmake, and Modules/Remote/Ultrasound.remote.cmake.

T-Rex T-Rex Logs

What T-Rex did

  • T-Rex fetched and checked out the old pin af317fbb92e8233fc00ba90f40d5f7b3eefd1560 and the new pin 5f8f4f8ec146991aac3a224e49aac7fc5ccd61a5, both with exit code 0.
  • T-Rex verified that the complete-history ancestry verification returned exit code 0, proving the old pin is an ancestor of the new pin.
  • T-Rex examined the build harness outputs to confirm the executed command, working directory, and exit code were captured.
  • T-Rex cataloged the artifacts produced by the run, including several log artifacts and two shell-script artifacts, to support reviewer inspection.

View all artifacts

T-Rex Ran code and verified through T-Rex

Important Files Changed

Filename Overview
Modules/Remote/CudaCommon.remote.cmake Advances the CudaCommon pin; CUDA-specific compilation depends on CI coverage.
Modules/Remote/IOOpenSlide.remote.cmake Advances the IOOpenSlide pin; OpenSlide-dependent compilation depends on CI coverage.
Modules/Remote/Ultrasound.remote.cmake Advances the Ultrasound pin; its VTK-dependent configuration depends on CI coverage.
Modules/Remote/BioCell.remote.cmake Advances the BioCell pin validated by the reported local Release build.
Modules/Remote/HASI.remote.cmake Advances the HASI pin validated by the reported local Release build.
Modules/Remote/PerformanceBenchmarking.remote.cmake Advances the PerformanceBenchmarking pin validated by the reported local Release build.
Modules/Remote/SCIFIO.remote.cmake Advances the SCIFIO pin validated by the reported local Release build.
Modules/Remote/Shape.remote.cmake Advances the Shape pin validated by the reported local Release build.
Modules/Remote/SkullStrip.remote.cmake Advances the SkullStrip pin validated by the reported local Release build.
Modules/Remote/VkFFTBackend.remote.cmake Advances the VkFFTBackend pin validated by the reported local Release build.
Modules/Remote/WebAssemblyInterface.remote.cmake Advances the WebAssemblyInterface pin validated by the reported local Release build.

Reviews (1): Last reviewed commit: "COMP: Advance remote-module pins to upst..." | Re-trigger Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Remotes Issues affecting the Remote module type:Compiler Compiler support or related warnings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant