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 compsets and tests to use clm6_0, once CAM moves to ctsm5.2.0 or later tag #1022

Closed
ekluzek opened this issue May 1, 2024 · 5 comments
Assignees
Labels
CoupledEval3 enhancement New feature or request

Comments

@ekluzek
Copy link

ekluzek commented May 1, 2024

What is the feature/what would you like to discuss?

Compsets and tests currently using CLM51 should be updated to CLM60 once CAM uses ctsm5.2.0 or later versions of CTSM. ctsm5.2.0 includes new datasets and hence will change answers from previous ctsm5.1 tags. The CTSM issue about removal of CLM51 is here:

ESCOMP/CTSM#2379

Is there anyone in particular you want to be part of this conversation?

No response

Will this change (regression test) answers?

Yes

Will you be implementing this enhancement yourself?

Any CAM SE can do this

@ekluzek ekluzek added the enhancement New feature or request label May 1, 2024
@cacraigucar
Copy link
Collaborator

cacraigucar commented May 6, 2024

@ekluzek - Is this a requirement when we move to ctsm5.2.0, or can it be done in phases? I'm asking because I was planning on making a quick CAM tag which matches beta17. I was planning on just updating the externals, but this will bring in ctsm5.2.0. Do I need to make a change to all of our tests with that tag as well? The reason I'm making this CAM tag is in preparation for Jim bringing in git-fleximod and having CAM be non-answer changing with his mods.

@cacraigucar
Copy link
Collaborator

I should also mention that we do not have any tests which are using CLM52, they are all using either CLM50 or CLM51 depending on what we are testing.

@ekluzek
Copy link
Author

ekluzek commented May 6, 2024

@cacraigucar thanks for the discussion. This is NOT a requirement for using ctsm5.2.0 in CAM. I suggest you do this in phases as you propose. Using CLM50 or CLM51 for now is fine. And it's probably best to NOT clutter the git fleximod update with changes like this. Getting it into the beta tag AFTER git fleximod does sound good to me though...

@cacraigucar
Copy link
Collaborator

Thanks for the confirmation that we can still use CLM50 and CLM51 for testing purposes. I am not going to put a calendar timeline on when we phase out the old testing, as we are currently concentrating on creating our CESM3 compsets and tests. Those WILL use CLM60. We expect this phase to take some time as "the devil is in the details". After that phase, we will be looking at current older testing and either eliminating or updating tests to use CAM7. At that time, we will also update tests to use CLM60. So our timing on when this is done will be event driven instead of calendar driven.

@brian-eaton
Copy link
Collaborator

This was resolved in the cam6_4_011 tag during the update to cesm3_0_alpha02a submodules. CLM60 is now used in all compsets using CAM70.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CoupledEval3 enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

3 participants