ESCOMP / POP2-CESM

Parallel Ocean Program (POP2) in CESM
http://www.cesm.ucar.edu/models/cesm2/ocean/
4 stars 24 forks source link

Update marbl #75

Closed mnlevy1981 closed 1 year ago

mnlevy1981 commented 1 year ago

Description of changes:

Update MARBL to use latest version (after https://github.com/marbl-ecosys/MARBL/pull/423 is accepted / merged / tagged)

Testing:

Test case/suite: none yet; will run aux_pop and aux_pop_MARBL and comment on results below Test status: [bit for bit, roundoff, climate changing]

Fixes [POP2 Github issue #] No tagged issues, though there are some MARBL-related issues that might be easy to fix that could be included

User interface (namelist or namelist defaults) changes? latest+4p2z is a new valid value for OCN_BGC_CONFIG

mnlevy1981 commented 1 year ago

I still need to update Externals_POP.cfg once there is a MARBL tag.

Should we run 4p2z by default? I either need to add a 4p2z test to aux_pop_MARBL or make the default 4p2z and add a 3p1z test to aux_pop_MARBL depending on the answer to that question.

mnlevy1981 commented 1 year ago

Testing results: I ran aux_pop_MARBL on cheyenne (with both intel and gnu). All tests fail NLCOMP due to additional variables that were added to marbl_in in recent MARBL updates (we are jumping from marbl0.40.3 to marbl0.43.0, so there's a lot to catch up on). Further, all tests are failing BASELINE because those recent MARBL commits also added some new diagnostics for FEISTY. There are two tests that are actually changing answers, and failing the BASELINE due to RMS differences detected by cprnc: