Closed eclare108213 closed 3 months ago
Will need some changes in Icepack documentation and maybe add a test case. Also need to update the icepack interface documentation (run ./icepack.setup --docintfc). CICE will also eventually need a new namelist added.
See also https://github.com/CICE-Consortium/CICE/pull/965
This PR and the CICE PR are ready to merge assuming we are happy leaving the old 'two-step' as the default. Please confirm that's how we want to proceed. We can always change the default later.
I did do a fully-coupled test here with the one-step option and the solutions are quite different. Climate changing? Maybe not, but I think this is the correct way to go. We will definitely be using this in CESM3.
Draft for Plante et al. to confirm that these changes are all that are needed, separate from their flooding/snow-ice and forcing modifications. I still need to add tests to the test suite and update the Icepack documentation, then implement and test in CICE. Suggestions for better congel_freeze option labels are welcome.
I realigned some columns while working on this. Ignore white space when looking at diffs!
congel_freeze = ‘two-step’
is BFB in the base_suite -- this is currently the default (as before).congel_freeze = 'one-step'
is not BFB. Settingconserv_check = .true.
does not turn up any issues. A QC comparison for CICE PR #965 shows the differences betweencongel_freeze = 'one-step'
and'two-step'
are not climate changing in CICE standalone simulations.congel_freeze = 'one-step'
Namelist flag congel_freeze chooses which formulation to use. The original is ‘two-step’, since only the mushy boundary moves in the first step and the phase change happens in the next step. Plante et al. (‘one-step’) moves the boundary and performs the phase change in a single time step.
Closes #481