v12: Remove support for mom5

[mathomp4]

This is one of a set of PRs to remove support for MOM5 in GEOSgcm v12. There are PRs in:

• GEOSgcm: v12: Updates for FMS 2025 Support, remove mom5 GEOSgcm#933
• GEOS_OceanGridComp: v12: Remove support for mom5  #100
• GEOSgcm_App: v12: Remove support for mom5 GEOSgcm_App#729
• FVdycoreCubed_GridComp: v12: Updates needed for FMS 2025 FVdycoreCubed_GridComp#308
• fvdycore: v12: Updates needed for FMS 2025 GFDL_atmos_cubed_sphere#112
• mom6: geos/v3.7

This will need updates for Dual Ocean support so we keep draft for now. The needed changes are in GEOS_OceanGridComp and in mom6.

At the moment there is the commented MOM5 dual ocean code in the MOM6 plug. We will work with @mfmehari and @atrayano and @sinakhani to get this in. This will need a put routine that @mfmehari has already created for his OBIO work.

ETA: Also, I believe @Dooruk needs dual ocean as well for his work!

[github-actions]

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.

[github-actions]

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.

[mathomp4]

@mfmehari From the CI, I guess this means we need an update to mom6 as well right? For the new ocean_model_put_prog_tracer call?

[Dooruk]

This might not be the best place to ask this but I see lot of relevant folks are tagged here. @amolod and I are wondering if there are any difference between v11 and v12 for the AOIL module, the answer is most likely no but wouldn't hurt to ask:

https://github.com/GEOS-ESM/GMAO_Shared/blob/main/GEOS_Shared/atmOceanIntLayer.F90

[mfmehari]

@mfmehari From the CI, I guess this means we need an update to mom6 as well right? For the new ocean_model_put_prog_tracer call?

@mathomp4 The routine is added now.

[mfmehari]

This is one of a set of PRs to remove support for MOM5 in GEOSgcm v12. There are PRs in:

• GEOSgcm: v12: Remove support for mom5 GEOSgcm#933
• GEOS_OceanGridComp: v12: Remove support for mom5  #100
• GEOSgcm_App: v12: Remove support for mom5 GEOSgcm_App#729
• mom6:

This will need updates for Dual Ocean support so we keep draft for now. The needed changes are in GEOS_OceanGridComp and in mom6.

At the moment there is the commented MOM5 dual ocean code in the MOM6 plug. We will work with @mfmehari and @atrayano and @sinakhani to get this in. This will need a put routine that @mfmehari has already created for his OBIO work.

ETA: Also, I believe @Dooruk needs dual ocean as well for his work!

Connections for dual-ocean in MOM6 are set, further updates need to be done in the other GCs for dual ocean to work ...

[sinakhani]

This might not be the best place to ask this but I see lot of relevant folks are tagged here. @amolod and I are wondering if there are any difference between v11 and v12 for the AOIL module, the answer is most likely no but wouldn't hurt to ask:

https://github.com/GEOS-ESM/GMAO_Shared/blob/main/GEOS_Shared/atmOceanIntLayer.F90

You may check it here (GEOS-ESM/GEOSgcm@v11.7.1...GCMv12-rc16). I do not see changes in atmOceanIntLayer.F90 file between v11.7.1 and GCMv12-rc16.

[Dooruk]

This might not be the best place to ask this but I see lot of relevant folks are tagged here. @amolod and I are wondering if there are any difference between v11 and v12 for the AOIL module, the answer is most likely no but wouldn't hurt to ask:
https://github.com/GEOS-ESM/GMAO_Shared/blob/main/GEOS_Shared/atmOceanIntLayer.F90

You may check it here (GEOS-ESM/[email protected]). I do not see changes in atmOceanIntLayer.F90 file between v11.7.1 and GCMv12-rc16.

Thanks @sinakhani, is GCMv12-rc16 the version you recommend me testing with? In other words with land mask and vertical mixing changes?

[github-actions]

This PR is being prevented from merging because you have not added one of our required labels: 0 diff, 0 diff trivial, Non 0-diff, 0 diff structural, 0-diff trivial, Not 0-diff, 0-diff, automatic, 0-diff uncoupled, github_actions. Please add one so that the PR can be merged.