COMP: Advance remote-module pins to upstream default-branch HEAD#6687
Open
hjmjohnson wants to merge 1 commit into
Open
COMP: Advance remote-module pins to upstream default-branch HEAD#6687hjmjohnson wants to merge 1 commit into
hjmjohnson wants to merge 1 commit into
Conversation
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).
hjmjohnson
marked this pull request as ready for review
July 23, 2026 02:43
Contributor
|
| 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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Advance 11 remote-module
GIT_TAGpins 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):
std::abs<double>(fix in COMP: Fix ill-formed std::abs<double> in ConditionalMedianImageFilter RTKConsortium/RTK#975).trx_cppsub-build cannot resolve ITK's Eigen/ZLIB (fix in COMP: Make ITK ZLIB/Eigen target resolution work on ITK v5 and v6 tee-ar-ex/ITKTractographyTRX#30).GetKernelMedialness) present at all pins.BUILD_TESTING=ON.