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

Rework list of compsets to align with the compsets needed for the ctsm6.0.0 release #2954

Open
ekluzek opened this issue Feb 6, 2025 · 1 comment
Labels
bfb bit-for-bit next this should get some attention in the next week or two. Normally each Thursday SE meeting. priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations science Enhancement to or bug impacting science usability Improve or clarify user-facing options

Comments

@ekluzek
Copy link
Collaborator

ekluzek commented Feb 6, 2025

We need to rework the list of compsets to align with the compsets that are needed for the ctsm6.0.0 release:

https://docs.google.com/spreadsheets/d/1nJl4lbbwBq60quyaERJ6OvVJxF4kDu4xX7zZenZgyB8/edit?gid=1789776230#gid=1789776230

In large degree we should leave the Clm45 and Clm50 compsets in place, but we should look for some to remove.

Currently there are 15 Clm45 compsets:

    <alias>I1PtClm45SpRs</alias>
    <alias>I2000Clm45BgcCropQianRs</alias>
    <alias>I1850Clm45BgcCrop</alias>
    <alias>I1850Clm45BgcCru</alias>
    <alias>IHistClm45BgcCrop</alias>
    <alias>IHistClm45BgcCropQianRs</alias>
    <alias>I2000Clm45Sp</alias>
    <alias>I2000Clm45BgcCrop</alias>
    <alias>I2000Clm45Fates</alias>
    <alias>I2000Clm45FatesRs</alias>
    <alias>I1850Clm45Bgc</alias>
    <alias>IHistClm45Bgc</alias>
    <alias>IHistClm45BgcCru</alias>
    <alias>IHistClm45Sp</alias>
    <alias>I2000Clm45VicCru</alias>

We should remove the Vic one for sure. And remove the Cru ones. We should probably support only the Rs or plain ones. And I think there is no reason to support both the Qian and GSWP3 versions.

The list of Clm50 compsets is longer having 54 of them. Again I think it makes sense to remove Cru. And we should remove the Rtm compsets and let the Clm45 compsets be the ones that work with Rtm. Other's should be examined for removal as well.

For Clm60 there's 37 compsets, and possibly some of the ones in the Clm50 list should be just for the latest Clm60 compset list. Do we need both Cru and Qian forcing for Clm60 or just use the older physics for these? Qian is used to save download and runtime for single point sites, so that might still be useful for Clm60.

@ekluzek ekluzek added bfb bit-for-bit next this should get some attention in the next week or two. Normally each Thursday SE meeting. priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations science Enhancement to or bug impacting science usability Improve or clarify user-facing options labels Feb 6, 2025
@slevis-lmwg
Copy link
Contributor

#1895 will partly address this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bfb bit-for-bit next this should get some attention in the next week or two. Normally each Thursday SE meeting. priority: high High priority to fix/merge soon, e.g., because it is a problem in important configurations science Enhancement to or bug impacting science usability Improve or clarify user-facing options
Development

No branches or pull requests

2 participants