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

Redefine NHIST and N1850 to use Cloud2 tunings #515

Open
wants to merge 1 commit into
base: noresm2_3_develop
Choose a base branch
from

Conversation

gold2718
Copy link

The KeyClim Cloud2 tunings are now the default for NHIST and N1850 compsets

  • Added compset and test to reproduce the Cloud2 experiment
  • Add namelist definitions, defaults and exceptions for clubb_meltpt_temp and clubb_dt_low
  • Modify WBF effective factor for keyClim
  • Update CAM external tag
  • Added a top-level script, describe_version (taken from CESM), which documents the current code configuration.

   for clubb_meltpt_temp and clubb_dt_low
Modify WBF effective factor for keyClim
Update CAM external tag
@gold2718 gold2718 added this to the NorESM2.3 milestone Apr 22, 2024
@gold2718 gold2718 requested a review from mvertens April 22, 2024 13:30
@gold2718 gold2718 self-assigned this Apr 22, 2024
@oyvindseland
Copy link

While I can certainly take over the issue, I think the critical part is the review and not the actual merge. So in order to save time I think it is better if I review the code and Dirk or Steve do the merge @MichaelSchulzMETNO

Copy link

@oyvindseland oyvindseland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This pull request is just adding a new cloud 2 SSP585 data set. There is no redefinition as in the title of the issue. Is it only a wrong title?

Copy link

@oyvindseland oyvindseland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on discussion with Dirk Olivie I think that this pull request is not relevant any more and can be cancelled. Given that the cloud2 settings should be default also the SSP585 should use the same settings as NHIST

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants