Add nightly HIP build CI workflow#777
Merged
Merged
Conversation
fc9cac6 to
6082000
Compare
612ef00 to
09bad55
Compare
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.
What
Adds
.github/workflows/hip-nightly.yml: a nightly (04:00 UTC) + manually dispatchable GitHub Actions job that builds ALIEN through the ROCm/HIP path (-DUSE_HIP=ON), complementing the existing CUDAlinux-buildandwindows-buildworkflows.Why
The HIP compatibility layer (
hip/,USE_HIPCMake wiring) is only exercised manually today. A nightly build surfaces regressions in that path before they land.How
ubuntu-24.04runner (no AMD GPU) → offline compile only;CMAKE_HIP_ARCHITECTURESis pinned (gfx1100) sinceenable_language(HIP)cannot auto-detect an arch without a GPU.repo.radeon.comapt repo (rocm-hip-runtime-dev— HIP compiler + headers + device libs, no driver needed).pipx(the project requires it; 24.04 ships older).ninjapreset +-DUSE_HIP=ON -DCMAKE_PREFIX_PATH=/opt/rocm, exactly the invocation documented in the README.EngineInterfaceTests,NetworkTests) as a smoke test; the GPU-boundEngineTestsis skipped (no AMD GPU on the runner).windows-ci.yml: skips when there were no commits in the last 24h; frees runner disk before the heavy vcpkg + ROCm install.Not verified locally
No ROCm/AMD hardware was available to run this — the workflow is unverified end-to-end. The first CI run may need tweaks to the exact ROCm package set, the pinned
HIP_ARCH, or disk headroom.🤖 Generated with Claude Code