Skip to content

Updates needed for MOM6 PR1708 and PR1712 - #142

Merged
mathomp4 merged 3 commits into
developfrom
feature/update-mom6-3.10
Oct 13, 2026
Merged

mathomp4 merged 3 commits into
developfrom
feature/update-mom6-3.10

Conversation

@mathomp4

@mathomp4 mathomp4 commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

This PR has updates needed for moving to mom6 geos/v3.10 which has updates from:

That first one is non-zero-diff in that:

  1. CAu and CAv will change in MOM.res.nc (Per this comment: "Once again, we DO expect there to be changes in the restart files in the CAu and CAv fields at the coastlines, but these do not impact the exact reproducibility across restarts, nor do they change any solutions.")
  2. in ocean_geometry.nc, there are now geolatu, geolatv, geolonu, and geolonv

I'm labeling as non-zero-diff for coupled then. Note: The changes here are only to allow building with mom6 geos/v3.10, but we track things "together"


Update

Interesting:

Comparing ocean_geometry.nc...
Running /home/mathomp4/local-SLES15/bin/nccmp -dfsB  stock-2026Oct13-1day-c12-MOM6//scratch/ocean_geometry.nc mom310-2026Oct13-1day-c12-MOM6//scratch/ocean_geometry.nc
Success!

I do not see the ocean_geometry.nc change in my c12 v14 test. Not sure why.

@mathomp4 mathomp4 added 0 diff dataocean Zero-diff for the dataocean model non 0-diff coupled Non-zero-diff for coupled model non 0-diff dataatm Non-zero-diff for dataatm labels Oct 13, 2026
@mathomp4

Copy link
Copy Markdown
Member Author

NOTE: CI will probably fail for chicken-and-egg issues. Once I confirm this branch and geos/v3.10 build together, I'll make a PR in GEOSgcm which will be an arbiter.

@mathomp4
mathomp4 marked this pull request as ready for review October 13, 2026 17:44
@mathomp4
mathomp4 requested review from a team as code owners October 13, 2026 17:44
@mathomp4
mathomp4 merged commit 8f8d117 into develop Oct 13, 2026
35 of 40 checks passed
@mathomp4
mathomp4 deleted the feature/update-mom6-3.10 branch October 13, 2026 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff dataocean Zero-diff for the dataocean model non 0-diff coupled Non-zero-diff for coupled model non 0-diff dataatm Non-zero-diff for dataatm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants