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

Post cesm2_3_alpha17a Updates #24

Merged
merged 5 commits into from
Feb 4, 2024
Merged

Post cesm2_3_alpha17a Updates #24

merged 5 commits into from
Feb 4, 2024

Conversation

gdicker1
Copy link
Contributor

@gdicker1 gdicker1 commented Jan 29, 2024

This PR moves CAM up to tag cam6_3_145 while adding a patch for NVHPC compilers, moving CIME up to tag cime6.0.0182, and brings in recent changes to mpas-seaice that fix Intel OneAPI builds.

@gdicker1 gdicker1 marked this pull request as ready for review January 30, 2024 17:54
@gdicker1
Copy link
Contributor Author

Since #22 has been merged, this is now moved out of draft. This PR should be merged quickly, dependent on tests. I skipped most tests in #22 since I expected CAM would break without the patches in EWOrg/CAM #12

@gdicker1 gdicker1 changed the title DRAFT: Post cesm2_3_alpha17a Updates Post cesm2_3_alpha17a Updates Jan 30, 2024
@gdicker1
Copy link
Contributor Author

@dazlich, could you test this branch for the FullyCoupled compset?

I got an error I really didn't expect when trying to run ./case.build (with any compiler). Essentially the paths don't look right to me and there's a mention of 'Can't locate Switch.pm in @INC ...'

Excerpt of error:

2024-01-29 19:40:32 ice
Create namelist for component mpassi
Calling /glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/EW_tst2/components/mpas-seaice//cime_config/buildnml
ERROR: Command: '/glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/EW_tst2/components/mpas-seaice/bld/build-namelist -infile /glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/cases/2024Jan29_172743_EWTest_postalpha17a_FullyCoupled.mpasa120.derecho.gnu/Buildconf/mpassiconf/cesm_namelist -caseroot /glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/cases/2024Jan29_172743_EWTest_postalpha17a_FullyCoupled.mpasa120.derecho.gnu -casebuild /glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/cases/2024Jan29_172743_EWTest_postalpha17a_FullyCoupled.mpasa120.derecho.gnu/Buildconf -cimeroot /glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/EW_tst2/cime -inst_string '' -decomp_prefix 'mpas-o.graph.info.QU120.' -date_stamp '230524' -ice_grid 'oQU120' -ice_ic_mode 'cold_start' -ice_bgc 'no_ice_bgc' -surface_mode 'free' -iceberg_mode 'none' -prognostic_mode 'full' -ntasks_ice '128' -ninst_ice '1'' failed with error 'Can't locate Switch.pm in @INC (you may need to install the Switch module) (@INC entries checked: /glade/u/apps/derecho/23.09/opt/perl/lib/perl5/x86_64-linux-thread-multi /glade/u/apps/derecho/23.09/opt/perl/lib/perl5 //////////////////////////////////////////////////////////////////////////////////////glade/u/apps/derecho/23.09/spack/opt/spack/perl/5.38.0/gcc/7.5.0/jqrn/lib/perl5 //////////////////////////////////////////////////////////////////////////////////////glade/u/apps/derecho/23.09/spack/opt/spack/perl/5.38.0/gcc/7.5.0/jqrn/lib/site_perl/5.38.0/x86_64-linux-thread-multi //////////////////////////////////////////////////////////////////////////////////////glade/u/apps/derecho/23.09/spack/opt/spack/perl/5.38.0/gcc/7.5.0/jqrn/lib/site_perl/5.38.0 //////////////////////////////////////////////////////////////////////////////////////glade/u/apps/derecho/23.09/spack/opt/spack/perl/5.38.0/gcc/7.5.0/jqrn/lib/5.38.0/x86_64-linux-thread-multi //////////////////////////////////////////////////////////////////////////////////////glade/u/apps/derecho/23.09/spack/opt/spack/perl/5.38.0/gcc/7.5.0/jqrn/lib/5.38.0) at /glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/EW_tst2/components/mpas-seaice/bld/build-namelist line 1603.
BEGIN failed--compilation aborted at /glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/EW_tst2/components/mpas-seaice/bld/build-namelist line 1603.' from dir '/glade/work/gdicker/EarthWorks/EWRepo_PullRequests/2024Jan23_TagUpdate_cesm2_3_alpha17a/cases/2024Jan29_172743_EWTest_postalpha17a_FullyCoupled.mpasa120.derecho.gnu/Buildconf/mpassiconf'
ERROR: case.build failed

@dazlich
Copy link
Contributor

dazlich commented Jan 30, 2024 via email

@gdicker1
Copy link
Contributor Author

@dazlich If you've already fetched the code, you might need to re-do it1. I was pointing mpassi-ew2.0.001 to the wrong commit. I've fixed/pushed that tag now.

Footnotes

  1. Or at least git fetch origin && git checkout mpassi-ew2.0.001 in the mpas-seaice directory.

Get changes for oQU grid default values
@dazlich
Copy link
Contributor

dazlich commented Jan 30, 2024 via email

@gdicker1
Copy link
Contributor Author

Ah darn, I was hoping I was just setting something up incorrectly. Do you think you can handle this error @dazlich?

@dazlich
Copy link
Contributor

dazlich commented Jan 30, 2024 via email

@dazlich
Copy link
Contributor

dazlich commented Jan 30, 2024 via email

@gdicker1
Copy link
Contributor Author

gdicker1 commented Feb 3, 2024

I was able to:

  • fetch and checkout_externals this code
  • get 120km FHS94, FKESSLER, QPC6, F2000climo, and FullyCoupled to setup, build, and run

Depending on your thoughts or tests, I think I'm good to merge this @dazlich.

This gets us a lot of the way to a EWMv2.1 release. Next we'll need the branch and changes to incorporate MPAS-A OpenACC!

@areanddee
Copy link
Contributor

areanddee commented Feb 3, 2024 via email

@dazlich
Copy link
Contributor

dazlich commented Feb 3, 2024 via email

@gdicker1 gdicker1 merged commit 28dc50c into develop Feb 4, 2024
@gdicker1 gdicker1 deleted the update/post_alpha17a branch April 23, 2024 23:16
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.

3 participants