Skip to content

Update issm package requirement to specific commit#22

Closed
lawrenceabird wants to merge 7 commits into
mainfrom
lawrenceabird-upstream-latest
Closed

Update issm package requirement to specific commit#22
lawrenceabird wants to merge 7 commits into
mainfrom
lawrenceabird-upstream-latest

Conversation

@lawrenceabird

@lawrenceabird lawrenceabird commented Oct 10, 2025

Copy link
Copy Markdown
Collaborator

Testing PR update process with upstream ISSM changes


🚀 The latest prerelease access-issm/pr22-6 at b41b5bd is here: #22 (comment) 🚀

Testing PR update process with upstream ISSM changes
@github-actions

Copy link
Copy Markdown

🚀 Attempted to deploy access-issm Prerelease pr22-1 with commit 1f1c025

🖥️ Gadi Deployment ❌

Comment thread spack.yaml
Co-authored-by: Aidan Heerdegen <aidan.heerdegen@gmail.com>
@github-actions

Copy link
Copy Markdown

🚀 Attempted to deploy access-issm Prerelease pr22-2 with commit c88e885

🖥️ Gadi Deployment ❌

@github-actions

Copy link
Copy Markdown

🚀 Attempted to deploy access-issm Prerelease pr22-3 with commit b3c91d6

🖥️ Gadi Deployment ❌

@github-actions

Copy link
Copy Markdown

🚀 Attempted to deploy access-issm Prerelease pr22-4 with commit fd2175f

🖥️ Gadi Deployment ❌

@aidanheerdegen

aidanheerdegen commented Oct 10, 2025

Copy link
Copy Markdown
Member

As a sanity check I made a PR that didn't change the code, but did update the spack-packages version (to pick up @Justin Kin Jun Hew's most recent changes) and added the issm package to the module injection

#24

That built ok. See the comment that shows how to use it:

#24 (comment)

Following those instructions:

$ module use /g/data/vk83/prerelease/modules
$ module load access-issm/pr24-1
Loading access-issm/pr24-1
  Loading requirement: access-triangle/1.6.1-iju5vxs access-issm/dependencies/pr24-1/openmpi/4.1.7-3enketu
    access-issm/dependencies/pr24-1/issm/2025.04.11-lsvbd66

You can see above it loaded the issm dependent module. So if we do module show on that:

$ module show access-issm/dependencies/pr24-1/issm/2025.04.11-lsvbd66
-------------------------------------------------------------------
/g/data/vk83/prerelease/apps/spack/0.22/release/modules/linux-rocky8-x86_64/access-issm/dependencies/pr24-1/issm/2025.04.11-lsvbd66:

module-whatis   {Ice-sheet and Sea-Level System Model.}
module          load access-triangle/1.6.1-iju5vxs
module          load access-issm/dependencies/pr24-1/openmpi/4.1.7-3enketu
prepend-path    PATH /g/data/vk83/prerelease/apps/spack/0.22/release/linux-rocky8-x86_64/gcc-13.2.0/issm-git.2025.04.11_2025.04.11-lsvbd66npxugfbxsmljnsfr6cqovvcoy/bin
prepend-path    CMAKE_PREFIX_PATH /g/data/vk83/prerelease/apps/spack/0.22/release/linux-rocky8-x86_64/gcc-13.2.0/issm-git.2025.04.11_2025.04.11-lsvbd66npxugfbxsmljnsfr6cqovvcoy/.
-------------------------------------------------------------------

So it adds the bin to PATH, but if we look in the matching lib:

$ ls /g/data/vk83/prerelease/apps/spack/0.22/release/linux-rocky8-x86_64/gcc-13.2.0/issm-git.2025.04.11_2025.04.11-lsvbd66npxugfbxsmljnsfr6cqovvcoy/lib/
BamgConvertMesh_python.so       InterpFromMesh2d_python.so        libISSMApi_python.so.0.0.0  libISSMPython.so
BamgMesher_python.so            InterpFromMeshToGrid_python.so    libISSMCore.a               libISSMPython.so.0
BamgTriangulate_python.so       InterpFromMeshToMesh2d_python.so  libISSMCore.so              libISSMPython.so.0.0.0
ContourToMesh_python.so         InterpFromMeshToMesh3d_python.so  libISSMModules.a            MeshPartition_python.so
ContourToNodes_python.so        IssmConfig_python.so              libISSMModules.so           MeshProfileIntersection_python.so
ElementConnectivity_python.so   libISSMApi_python.a               libISSMOverload.a           NodeConnectivity_python.so
ExpToLevelSet_python.so         libISSMApi_python.so              libISSMOverload.so          ProcessRifts_python.so
InterpFromGridToMesh_python.so  libISSMApi_python.so.0            libISSMPython.a             Triangle_python.so
$

So next step is to either modify the package to set an environment variable to point lib, or do it in the deployment repo spack.yaml.

Use full commit hash and update tag for new version
@github-actions

Copy link
Copy Markdown

🚀 Attempted to deploy access-issm Prerelease pr22-7 with commit f688adc

🖥️ Gadi Deployment ❌

@github-actions

Copy link
Copy Markdown

🚀 Attempted to deploy access-issm Prerelease pr22-5 with commit 0101f92

🖥️ Gadi Deployment ❌

@lawrenceabird lawrenceabird deleted the lawrenceabird-upstream-latest branch October 16, 2025 20:38
@github-actions

Copy link
Copy Markdown

🚀 Attempted to deploy access-issm Prerelease pr22-6 with commit b41b5bd

🖥️ Gadi Deployment ❌

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.

2 participants