-
Notifications
You must be signed in to change notification settings - Fork 10
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
Create a new CTSM branch for "NorESM_LandSites_Platform" #5
Comments
Great that you have moved this forward @huitang-earth. About name of the new branch I think "noresm_landsites" is nice and simple. But most importantly, I think we should have the same name across the different repositories. |
How different will the new CTSM branch be from the ESCOMP CTSM master? If it is very similar, will it make sense to fetch updates from the ESCOMP repository and update master and other key branches in our fork so that the new local CTSM branch stays reasonably close to the forked master? I guess this can be done at a later stage, but I think it is important to think through this. A similar question is relevant for the issue NorESMhub/cime#20. |
Agreed, @matsbn. A new brach only makes sense if there are specific changes to CTSM for this platform. Otherwise, I agree that we should just keep master up to date. This is probably different than with cime, where there are in deed a lot of changes specific to this platform. |
@matsbn @kjetilaas Thanks for the comments. Yes, I am planning to separate the configuration or namelist changes from CTSM, so that we could directly use the master branch of CTSM. The configuration or namelist files will then be kept in "NorESM_LandSites_Platform" and copied from there to the corresponding folder during the running phase. We will try to do the same for CIME. Hopefully, it will work. :) |
Awesome! 💯 @huitang-earth will the change you made in src/utils/clmfates_interfaceMod.F90 be reported back to ESCOMP/CTSM (huitang-earth@ff424d1)? |
Yes. It has been reported back. But it has not been fixed yet: ESCOMP#1248 |
@matsbn @kjetilaas, I have taken out all the configuration changes for the platform in CTSM, and keep them as separate files in the "NorESM_LandSItes_Platform" repo. Please see the PR (NorESMhub/noresm-land-sites-platform#2) So, we don't need to create a new branch of CTSM and CIME for the land sites platform now. But do you want me to update the "master" branch of NorESMhub/CTSM to be syn with the ESCOMP/CTSM master? I can do it quickly if needed. |
It would be good to have a short description of all major branches in the NorESMhub/CTSM repo in a readme.md file, which is in the master branch on the CTSM repo. I have recently added the branch release-cesm2.2 to have the CTSM for an update to CESM/NorESM2.2 . This should contain the latest necessary for CESM/NorESM2.2. The question might be what the master in the NorESMhub/CTSM repo shall be used for. I would suggest this is a discussion and decision for the NorESM-CTSM core group. |
I have the same concern on "how the master in the NorESMhub/CTSM repo shall be used for". It can be (1) the "master" branch that always syn with ESCOMP CTSM without any changes, (2) the "master" branch of CTSM used by NorESM, in which we will keep track of the version used by NorESM. Hope to have a more thorough discussion. |
I suggest that you have a CTSM core telecon, I can join in. |
|
Perhaps, we could also discuss how we are going to merge this pull request #6 for CTSM by the way. Do we need to push these changes back to ESCOMP/CTSM?. It seems to be useful for the wider CESM/CLM community. |
Merge up to ctsm5.2.006 and deconflict
This issue is related to the issue 205 in NorESM repository and issue 20 in CIME repository.
The new CTSM branch will be used to keep the newest version of CTSM from NCAR that works with the most recent version of other model component (e.g., FATES). Note that whether this branch works with the atmosphere and ocean model components of NorESM is not tested.
The new CTSM branch will also contain a lot of setup for running single-cell simulations over observation sites.
The new CTSM branch is ready at my local repository. I can make a pull request if we decide on the name of the new branch.
What would be a proper name for the new CTSM branch? @annefou, @kjetilaas, @sunnivin, @DirkOlivie, @matsbn, @MichaelSchulzMETNO
The text was updated successfully, but these errors were encountered: