Skip to content

Port OVP.F90 off MAPL_GenericMod to MAPL3#419

Merged
mathomp4 merged 2 commits into
release/MAPL-v3from
feature/418-port-ovp-off-mapl-generic
May 1, 2026
Merged

Port OVP.F90 off MAPL_GenericMod to MAPL3#419
mathomp4 merged 2 commits into
release/MAPL-v3from
feature/418-port-ovp-off-mapl-generic

Conversation

@tclune
Copy link
Copy Markdown
Collaborator

@tclune tclune commented Apr 30, 2026

Summary

  • Closes Port OVP.F90 off MAPL_GenericMod (MAPL_MetaComp, MAPL_GetObjectFromGC) #418
  • Replace use MAPL2 with use MAPL + use mapl3g_GridGet
  • Replace MAPL_MetaComp/MAPL_GetObjectFromGC/MAPL_Get(STATE, LONS=...) with MAPL_GridCompGet(GC, grid=grid) + GridGetCoordinates(grid, LONS, lats)
  • Change LONS from REAL(ESMF_KIND_R4) to REAL(ESMF_KIND_R8) pointer (ESMF grid coords are R8)
  • Change OVP_mask dummy LONS from REAL to REAL(ESMF_KIND_R8)
  • Add MAPL.geom to CMake dependencies

Note: callers of OVP_init in GEOSgcm_GridComp (TR, GMI, ChemEnv) are currently commented out in CMake and will need updating when they are ported.

Part of MAPL #4730.

- Replace use MAPL2 with use MAPL and use mapl3g_GridGet
- Replace MAPL_MetaComp/MAPL_GetObjectFromGC/MAPL_Get with
  MAPL_GridCompGet + GridGetCoordinates
- Change LONS from REAL(R4) to REAL(R8) pointer (ESMF grid coords are R8)
- Add MAPL.geom to CMake dependencies

Closes #418
@tclune tclune requested review from a team as code owners April 30, 2026 22:54
@tclune tclune added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Apr 30, 2026
@mathomp4
Copy link
Copy Markdown
Member

mathomp4 commented May 1, 2026

Grah. Let me fix the CI for spack. Dang it.

@mathomp4 mathomp4 added the Skip Changelog Skips the Changelog Enforcer (use only when appropriate!) label May 1, 2026
@mathomp4 mathomp4 merged commit 59a3304 into release/MAPL-v3 May 1, 2026
16 of 18 checks passed
@mathomp4 mathomp4 deleted the feature/418-port-ovp-off-mapl-generic branch May 1, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch. Skip Changelog Skips the Changelog Enforcer (use only when appropriate!)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants