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

Rename chemistry compsets #1158

Open
fvitt opened this issue Oct 1, 2024 · 1 comment
Open

Rename chemistry compsets #1158

fvitt opened this issue Oct 1, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@fvitt
Copy link

fvitt commented Oct 1, 2024

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

Adjust chemistry compset names to adopt the new naming convention.
The document here
https://docs.google.com/spreadsheets/d/1nJl4lbbwBq60quyaERJ6OvVJxF4kDu4xX7zZenZgyB8/edit?pli=1&gid=1387633814#gid=1387633814

specifies "%CT4S" chemistry, while we currently we have "%CCTS4". Other chemistry compsets should be named consistently.

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

@tilmes

Will this change (regression test) answers?

Yes

Will you be implementing this enhancement yourself?

Yes

@fvitt fvitt added the enhancement New feature or request label Oct 1, 2024
@fvitt fvitt self-assigned this Oct 2, 2024
@fvitt
Copy link
Author

fvitt commented Oct 2, 2024

Current cam-chem compsets:

   FC2000climo          : 2000_CAM60%CCTS1_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FC2000climo_HCO      : 2000_CAM60%CCTS1%HEMCO_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FC2010climo          : 2010_CAM60%CCTS1_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FC2010climo_HCO      : 2010_CAM60%CCTS1%HEMCO_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCHIST               : HIST_CAM60%CCTS1_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCLTHIST             : HIST_CAM70%LT%CCTS1_CLM60%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCMTHIST             : HIST_CAM70%MT%CCTS1_CLM60%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCts4MTHIST          : HIST_CAM70%MT%CCTS4_CLM60%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCHIST_HCO           : HIST_CAM60%CCTS1%HEMCO_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCnudged             : HIST_CAM60%CCTS1%NUDG_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCts2nudged          : HIST_CAM60%CCTS2%NUDG_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCnudged_HCO         : HIST_CAM60%CCTS1%NUDG%HEMCO_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCts2nudged_HCO      : HIST_CAM60%CCTS2%NUDG%HEMCO_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCSD                 : HIST_CAM60%CCTS1%SDYN_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCts2SD              : HIST_CAM60%CCTS2%SDYN_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
   FCSD_HCO             : HIST_CAM60%CCTS1%SDYN%HEMCO_CLM50%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV

The proposed changes are

  • rename chemistry specifier "%CCTSn" as "%CTnS" in the long name (both CAM6 and CAM7)
  • rename the short names for CAM7 to be consistent with B compset short names proposed in the above document
    So CAM7 compset will be:
  FHISTC_LTt1s          : HIST_CAM70%LT%CT1S_CLM60%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
  FHISTC_MTt1s          : HIST_CAM70%MT%CT1S_CLM60%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV
  FHISTC_MTt4s          : HIST_CAM70%MT%CT4S_CLM60%SP_CICE%PRES_DOCN%DOM_MOSART_SGLC_SWAV

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

No branches or pull requests

1 participant