search

ESCOMP / POP2-CESM

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

fixed ndep problem for pop in nuopc cap #54

Closed mvertens closed 3 years ago

mvertens commented 3 years ago

Description of changes:

This only effects the nuopc cap. The problem was that separate ndep fields were advertised (Faxa_nhx, Faxa_nox) and what should have been advertised is a single field (Faxa_ndep) with 2 undistributed dimensions for nhx and noy.

Testing:

The following test were run and compared using cesm2_3_alpha03a and this pop PR code:

SMS_Vmct_Ln6.f19_g17.1850_CAM60%WCTS_CLM50%BGC-CROP_CICE_POP2%ECO%NDEP_MOSART_SGLC_WW3.cheyenne_intel.pop-default.validate
SMS_Vnuopc_Ln6.f19_g17.1850_CAM60%WCTS_CLM50%BGC-CROP_CICE_POP2%ECO%NDEP_MOSART_SGLC_WW3.cheyenne_intel.pop-default.validate

Test case/suite: Only test above was run Test status: All other configurations should be bfbf

Fixes: None

User interface (namelist or namelist defaults) changes? No

mvertens commented 3 years ago

@alperaltuntas @mlevy - it would be really helpful to get this into the next prealpha tag to resolve several failed nuopc tests.

mvertens commented 3 years ago

@alperaltuntas - thank you! Can you please add this to the plans page for alpha05a?

alperaltuntas commented 3 years ago

@mvertens - this is now added to cesm2_3_alpha05a tag.

mvertens commented 3 years ago

Thank you!!!

On Wed, Jun 2, 2021 at 9:28 AM Alper Altuntas @.***> wrote:

@mvertens https://github.com/mvertens - this is now added to cesm2_3_alpha05a tag.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ESCOMP/POP2-CESM/pull/54#issuecomment-853124836, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB4XCE7JYRNNU7MMWC72EHTTQZE2LANCNFSM45HL3NFQ .

-- Mariana Vertenstein CESM Software Engineering Group Head National Center for Atmospheric Research Boulder, Colorado Office 303-497-1349 Email: @.***