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

Update submodules to cesm3_0_beta04 versions #2640

Open
ekluzek opened this issue Jul 11, 2024 · 11 comments · Fixed by #2853
Open

Update submodules to cesm3_0_beta04 versions #2640

ekluzek opened this issue Jul 11, 2024 · 11 comments · Fixed by #2853
Assignees
Labels
done Issues whose closing PR is done but not yet merged (pending test re-run ok) enhancement new capability or improved behavior of existing capability

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented Jul 11, 2024

Update externals to those in cesm3_0_alpha02a for: cime, ccs_config, cmeps, share, mizuRoute, and cdeps

Right now this is:

cime6.1.0
ccs_config_cesm1.0.0
share1.1.2
cmeps1.0.2
cdeps1.0.43
cesm-coupling.n02_v2.1.3 (mizuroute)

#2590 updates cism, rtm, and mosart in addition to these (and updates cmeps and ccs_config as well). Between the two that should give us submodules for cesm3_0_alpha02a (and cesm3_0_beta02, as the other planned updates are only for CAM and MOM).

This will go on the b4b-dev branch.

@ekluzek ekluzek added enhancement new capability or improved behavior of existing capability bfb bit-for-bit labels Jul 11, 2024
@ekluzek ekluzek added this to the cesm3_0_beta02 milestone Jul 11, 2024
@ekluzek ekluzek self-assigned this Jul 11, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented Jul 11, 2024

Part of the reason to do this is because of the problems I saw in #2545.
The other is to keep up to date with cesm3_0 submodules in CESM tags

@ekluzek
Copy link
Collaborator Author

ekluzek commented Jul 11, 2024

Also note that @fischer-ncar found that a new ccs_config is needed for the setup to build, so at least that submodule will change. We'll let him figure this out first.

@ekluzek
Copy link
Collaborator Author

ekluzek commented Jul 17, 2024

This relates to #2654 another difference in cesm3_0_alpha02a is the removal of MCT and adding in:

libraries/mpi-serial for MPIserial_2.5.0

@ekluzek ekluzek added the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Jul 17, 2024
@slevis-lmwg slevis-lmwg removed the next this should get some attention in the next week or two. Normally each Thursday SE meeting. label Jul 18, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented Jul 24, 2024

I looked into this a bit, but it will take some effort to get the PF unit-testers to work with the removal of esmf_wrf_timemgr in the share code. This will bringing in some of the work that was done in the share code in

ESCOMP/CESM_share#48

esmf_wrf_timemgr will need to be removed from the CMakeLists.txt files for the ones that use it as well as the build for it at the top level. And the FindESMF cmake will need to be pointed to in the share directory, which might also mean that PIO will need to be made as well as it is in share code. We haven't had I/O in our unit tests at all to this point. But, this could be useful for example in @adrifoster functional unit testing that makes plots and needs I/O. For robustness it's good to have unit-tests that don't do I/O, but there are reasons it can be useful (for example it would be good to have unit-testing for the history modules).

Anyway, the main point here is that it will take some work to get this going. We'll probably need some help from Jim on this...

@wwieder @briandobbins

@slevis-lmwg
Copy link
Contributor

slevis-lmwg commented Oct 1, 2024

  • Reminder: This PR's update of ccs_config will allow us to also update this test from
    <test name="SMS_Ln9" grid="C96_t061" compset="I2000Clm60Sp" testmods="clm/clm60cam7LndTuningMode--clm/nofireemis">
    to
    <test name="SMS_Ln9" grid="C96_t232" compset="I2000Clm60Sp" testmods="clm/clm60cam7LndTuningMode--clm/nofireemis">
    ...as discussed in Change testlist_clm ne30pg3_t061 tests to ne30pg3_t232 #2799

@ekluzek ekluzek changed the title Update externals to cesm3_0_alpha02a versions Update submodules to cesm3_0_alpha02a versions Oct 2, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 2, 2024

As part of this we should update cmeps for this issue if a fix comes in:

ESCOMP/CMEPS#509

@samsrabin samsrabin moved this to In progress - master/b4b-dev in CTSM: Upcoming tags Oct 7, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 7, 2024

This will need to happen as part of cesm3_0_beta04 when the rpointer update comes in because we need to update to later versions of: cmeps and cime.

@wwieder
Copy link
Contributor

wwieder commented Oct 17, 2024

let's update the title and details here, presumably we want to update to the latest cesm_alpha tag?

@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 17, 2024

Pinging @billsacks

@ekluzek ekluzek changed the title Update submodules to cesm3_0_alpha02a versions Update submodules to cesm3_0_beta04 versions Oct 17, 2024
@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 25, 2024

Working on this again. Jim pointed me to versions of cime (6.1.37) and ccs_config (1.0.4) that work together and that helped me to get going on this.

There is still work to be done to get the full ESMF working for the unit testing, but that has gotten me started on the right path.

@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 29, 2024

OK, it looks like external requirements to get the PFUNIT tests to work are at odds with getting standard cases to work. To get standard cases to work I need to update share beyond share1.0.19, but the unit testing requires ESMF once I go to share1.0.20. So in order to get both working I need to get the PFUNIT tests to work with the full ESMF library. This is something we need to do anyway, but I was hopeful I could get something that would work in the meantime while I work on that.

@ekluzek ekluzek linked a pull request Nov 1, 2024 that will close this issue
@ekluzek ekluzek removed the bfb bit-for-bit label Nov 4, 2024
@ekluzek ekluzek added the done Issues whose closing PR is done but not yet merged (pending test re-run ok) label Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Issues whose closing PR is done but not yet merged (pending test re-run ok) enhancement new capability or improved behavior of existing capability
Projects
Status: Done (non release/external)
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants