Skip to content

Conversation

@blcc
Copy link

@blcc blcc commented Sep 27, 2024

Hi,
This PR add NorCPM as an ESP component.
It's won't effect any exist configuration.

Thanks,
Ping-Gin

@gold2718 gold2718 self-requested a review September 28, 2024 07:58
@gold2718 gold2718 added the enhancement New feature or request label Sep 28, 2024
Copy link

@gold2718 gold2718 left a comment

Choose a reason for hiding this comment

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

This PR needs a few tweaks. Also, please think about if NorCPM needs any help with archiving. For instance, the short-term archiver (<cime>/scripts/lib/CIME/case/case_st_archive.py) has to know which files to save after a successful run.
See especially <cime>/config/cesm/config_archive.xml where you can add filename patterns to look for and save.
We may need to tweak a couple of other files as well. We can figure that out after we have a test in NorESMhub/NorESM#565.

@blcc
Copy link
Author

blcc commented Sep 30, 2024

@gold2718 thanks.
I changed these two.

@blcc
Copy link
Author

blcc commented Oct 21, 2024

@gold2718
Hi,
Do you have some time to check this PR?
It is also related to NorESMhub/NorESM#565
Thanks.

@blcc blcc requested a review from gold2718 November 19, 2024 08:45
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

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants