Skip to content

Update to latest MARBL - #349

Merged
mnlevy1981 merged 17 commits into
ESCOMP:mainfrom
mnlevy1981:marbl_ver_update
Sep 9, 2026
Merged

Update to latest MARBL#349
mnlevy1981 merged 17 commits into
ESCOMP:mainfrom
mnlevy1981:marbl_ver_update

Conversation

@mnlevy1981

Copy link
Copy Markdown
Collaborator

We should not update MARBL until NCAR/MOM6#445 has been merged, otherwise turning on some MARBL diagnostics will lead to memory issues (currently MOM6 expects all 3D fields to have vertical dimension GV%ke but the new version of MARBL provides some diagnostics on vertical cell interfaces with length GV%ke+1)

klindsay28 and others added 14 commits July 11, 2026 08:31
remove MOM6_PASSIVE_TRACER_RESTART_OVERRIDE and rpointer file manipulation
correct DATM_MODE ALIGN variable
add more logging output
add MOM6_PHYS_CYCLE_MARBL_IC_FILE, MOM6_PHYS_CYCLE_USE_MARBL_IC_FILE
    to env_run.xml to initialize MARBL on firts PHYS_CYCLE cycle

set MOM6_PHYS_CYCLE_USE_MARBL_IC_FILE to FALSE in phys_cycle_postrun
…DE_POINTER

this parameter enables reading tracer ICs from multiple files

introduce MARBL_ENFORCE_TRACER_ZINT

set these parameters in phys_cycle_postrun

update MARBL IC specification in PHYS_CYCLE mode and RUN_TYPE == "hybrid"

remove unneeded Z2023 diagnostic coord
set AGE_IC_FILE_OVERRIDE_POINTER to MOM6_TRACER_IC_FILE_OVERRIDE_POINTER,
when it is set, or to rpointer file if RUN_TYPE=hybrid
@mnlevy1981 mnlevy1981 added this to the cesm3_0_alpha10a milestone Sep 8, 2026
@mnlevy1981

Copy link
Copy Markdown
Collaborator Author

There is one more MARBL PR as well (marbl-ecosys/MARBL#494) and then I'll take this out of draft mode

@mnlevy1981
mnlevy1981 marked this pull request as ready for review September 8, 2026 23:11
@mnlevy1981

Copy link
Copy Markdown
Collaborator Author

@alperaltuntas This will need an updated MOM6 tag, NCAR/MOM6#445 has been merged onto dev/ncar but I didn't make a tag for it. This is the last update I need for alpha10a, so once all of your PRs are merged this can be updated and merged and then tagged for the plans page.

@mnlevy1981

Copy link
Copy Markdown
Collaborator Author

I ran a GW1850MARBL_JRA test case with 1de42cb and verified that

(a) Variables moved to layer interfaces in NCAR/MOM6#445 and marbl-ecosys/MARBL#492 have the correct vertical dimension
(b) Parameters changed in marbl-ecosys/MARBL#494 are correct in marbl_in

I believe the MOM6 and MARBL tags provided here are what we want for cesm3_0_alpha10a, I'll merge this once the CI completes

@mnlevy1981
mnlevy1981 merged commit 75a527d into ESCOMP:main Sep 9, 2026
6 checks passed
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