Skip to content

Conversation

@matsbn
Copy link
Contributor

@matsbn matsbn commented Mar 12, 2025

Support for CGS units is removed, except for continuation from restart files written in CGS units.

Bit identical results have been confirmed with the NOIIAJRAOC compset and the tnx1v4 grid for the following cases:

  • Simulation from initial conditions with BLOM_VCOORD = "cntiso_hybrid".
  • Simulation from initial conditions with BLOM_VCOORD = "isopyc_bulkml"
  • Continuation from restart in MKS units.
  • Continuation from restart in CGS units.

milicak and others added 5 commits March 1, 2025 17:33
- Updated CGS units in comments to MKS units.
- In CIME configuration files, removed BLOM_UNITS and CGS specific namelist settings.
- Updated copyright statements.
@jmaerz
Copy link
Collaborator

jmaerz commented Mar 13, 2025

Hi @matsbn , could you briefly explain how to perform a simulation starting from cgs restart files?

@matsbn
Copy link
Contributor Author

matsbn commented Mar 13, 2025

Hi @matsbn , could you briefly explain how to perform a simulation starting from cgs restart files?

It is exactly the same as starting from a restart file in MKS units (either a branch run, hybrid run or specifying the restart file as the ocean initial condition by setting the ICFILE namelist variable). The unit system used for the restart file is automatically detected. As it is stated in mod_restart.F90:

"The ratio of bottom pressure to bottom geopotential should approximate sea water density that robustly can be tested if MKS or CGS units was used in the restart file."

@jmaerz jmaerz mentioned this pull request Mar 14, 2025
Copy link
Collaborator

@jmaerz jmaerz left a comment

Choose a reason for hiding this comment

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

Thanks for the explanation. From the iHAMOCC side, it looks good - I haven't checked all details in BLOM, but expect that it has been tested. Thanks!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in NorESM Development Mar 20, 2025
@matsbn matsbn merged commit 2f9f080 into master Mar 25, 2025
8 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in NorESM Development Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants