Skip to content

Updates for install-dir-in-experiment#1104

Draft
mathomp4 wants to merge 4 commits into
mainfrom
feature/experiment-install
Draft

Updates for install-dir-in-experiment#1104
mathomp4 wants to merge 4 commits into
mainfrom
feature/experiment-install

Conversation

@mathomp4

Copy link
Copy Markdown
Member

This is part of a suite of PRs to try and change how we handle GEOSgcm.x in GEOS. The issue is that more and more of GEOS subcomponents are becoming shared object libraries instead of static.

Overall Problem

That means the old trick of "copy someone else's GEOSgcm.x to my experiment" might not work. Why? Well, in GEOSgcm v11.10+, the Moist GridComp (for example) is now a shared object library and so just copying GEOSgcm.x isn't enough to actually use that other person's Moist GC code.

What we change here

In this repo we call a new cmake script that runs after install. It creates a "manifest" as it were. So, if a user has two experiments, you can compare the manifests that will be in the install to see what might have changed (not perfect, but better than nothing). For example, if you diff two manifests:

> diff installdir-v12-2026Jun23-1day-c48/install/GEOS_RUNTIME_MANIFEST.sha256 installdir-v12-2026Jun23-1day-c48-ChangeMoist-Try3/install/GEOS_RUNTIME_MANIFEST.sha256
190c190
< 01082779e76498ec996b15f7b06acd300384d7bb62b58091977c5204d1c14de1  9482728  lib/libGEOSmoist_GridComp.so
---
> b5f1b1ee19c614bf071f1d9004eec62dfcb1e021dc2a47ae2f0f1509d1bbf713  9482728  lib/libGEOSmoist_GridComp.so

@mathomp4 mathomp4 self-assigned this Jun 23, 2026
@mathomp4 mathomp4 added Contingent - DNA These changes are contingent on other PRs 0-diff agcm This pull request have verified to be zero-diff with the target branch in dataocean mode. 0-diff ogcm This pull request have verified to be zero-diff with the target branch in coupled mode. labels Jun 23, 2026
@github-actions

Copy link
Copy Markdown

This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged.

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

Labels

0-diff agcm This pull request have verified to be zero-diff with the target branch in dataocean mode. 0-diff ogcm This pull request have verified to be zero-diff with the target branch in coupled mode. Contingent - DNA These changes are contingent on other PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant