Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Once nuopc/cmeps cases work on izumi, add some testing #60

Open
billsacks opened this issue Aug 12, 2021 · 2 comments
Open

Once nuopc/cmeps cases work on izumi, add some testing #60

billsacks opened this issue Aug 12, 2021 · 2 comments

Comments

@billsacks
Copy link
Member

Once nuopc/cmeps cases work on izumi, I feel that we should add the following tests to the test suite:

  • NUOPC versions of all of the current izumi tests
  • Some Antarctica tests similar to existing Greenland tests; I'd suggest:
    • SMS_Vnuopc_D_Ly1_P24x1.f09_g17_ais8.T1850Ga.izumi_nag
    • SMS_Vnuopc_D_Ld5_P24x1.f10_f10_ais8_mg37.I1850Clm50SpGa.izumi_nag.cism-test_coupling
    • SMS_Vnuopc_D_Ly1_P24x1.f09_g17_ais8.T1850Ga.izumi_pgi
billsacks added a commit that referenced this issue Sep 29, 2021
Partially addresses #60 (Once nuopc/cmeps cases work
on izumi, add some testing).

I have not added any nuopc izumi_pgi tests, because I think that nuopc
isn't working with pgi yet.
billsacks added a commit that referenced this issue Sep 29, 2021
Partially addresses #60 (Once nuopc/cmeps cases work
on izumi, add some testing)
@billsacks
Copy link
Member Author

I have mostly addressed this in ba968a8 and 0974261. The remaining need is nuopc pgi tests, once pgi is working with nuopc: see the two comments referring to BUG(wjs, 2021-09-29, ESCOMP/CISM-wrapper#60) in testlist_cism.xml for details of the two tests that I feel should still be added.

@billsacks
Copy link
Member Author

Actually, I'm temporarily removing one other nuopc test that currently isn't working: ERS_D_Ly3.f09_g17_gris4.T1850Gg.cheyenne_intel.cism-noevolve. If I remember correctly, @mvertens has fixed this on a branch of CMEPS, so we should be able to add this test at some point. I have gone back and forth on whether it's worth having this test, but I currently think it could be helpful: even though a T compset noevolve configuration would be a weird thing to run for a production case, it's a convenient way to make sure that exact restart works for CISM in noevolve mode. The alternative would be to test this (i.e., restarting in noevolve mode) via an I compset, but running a multi-year I compset test takes significantly more time to turn around.

See 1d2d81b. We can back out that commit once this is working.

billsacks added a commit that referenced this issue Sep 29, 2021
Update externals (nuopc now the default!) and test list

Purpose of changes:

   - Update externals to versions tentatively planned for
     cesm2_3_beta06. A key change here is to make nuopc the default
     driver.

   - Rework test names now that nuopc is the default driver (so _Vmct is
     needed explicitly, _Vnuopc is not)

   - Add a few more nuopc tests, now that nuopc is working on izumi_nag
     (partially addresses
     #60)

   - Fix for build issue on izumi_nag
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

No branches or pull requests

1 participant