Skip to content

Conversation

@anton-seaice
Copy link
Collaborator

@anton-seaice anton-seaice commented Oct 29, 2025

This pull request merges the access-esm1.6 branch of CICE5 into the master branch.

The access driver is used for access-esm1.6, whilst the auscom driver is used for access-om2

pbd562 and others added 30 commits August 22, 2016 04:02
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_GC3_GA7@357 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
…models/cice_GC3_GA7_hxyo/

git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_GC3_GA7@358 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_GC3_GA7@359 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_GC3_GA7@360 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_GC3_GA7@366 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_GC3_GA7@367 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_GC3_GA7@368 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_GC3_GA7@372 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@393 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@394 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@395 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@396 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
… years.

git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@402 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@403 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
…tion errors.

git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@404 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@405 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
git-svn-id: file:///g/data/access/access-svn/cice/branches/access/cice_gsi8.1@406 f6bd92a4-46cf-401b-8a38-b7f7993d28bf
* Add build ci for access-esm1.6 branch
* Needed to support oneapi 2025 compiler.
Add license file from CICE-consortium
Ported https://github.com/ACCESS-NRI/cice4/blob/694a9fbd4ac29dc841b38aff002eb36da5b650f1/source/ice_history.F90#L2184-L2198 to cice5.

This avoids post-processing interpreting end of the time interval timestamps as the subsequent time interval.
Implement dump last in ESM1.6 driver
This makes the assumed salinity of sea ice configurable, so we can set it the same as the MOM value

(configured in MOM [here](https://github.com/ACCESS-NRI/access-esm1.6-configs/blob/88ac5aab5d6d2500209b7d610e68e5c2928222d4/ocean/input.nml#L367))

We use 4ppt in MOM for historical reasons, and did that in [CICE4](https://github.com/ACCESS-NRI/cice4/blob/694a9fbd4ac29dc841b38aff002eb36da5b650f1/source/ice_init.F90#L243) too.
It's hard to set this in the namelist because it is used at compile time to define several parameters

Set per MOM5 value:

https://github.com/ACCESS-NRI/FMS/blob/bf9b80423ea4f66efa389e03d3c19b26c009e8b6/constants/constants.F90#L90
* Expose ksno (thermal conductivity of snow ) to the namelist

Typical default value is 0.3, for CM2, use 0.2
Co-authored-by: Spencer Wong <88933912+blimlim@users.noreply.github.com>
@anton-seaice anton-seaice force-pushed the master-esm1.6-merge branch 3 times, most recently from a8c152b to 29a9581 Compare January 9, 2026 00:15
@anton-seaice
Copy link
Collaborator Author

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This file is changed to catch some improvements made to drivers/access/cpl_parameters.F90

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ice_ref_salinity, ksno and hs_min are moved to the namelist

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@anton-seaice
Copy link
Collaborator Author

@dougiesquire and @blimlim

Here, I present a monster PR for your enjoyment.

This merges the access-esm1.6 branch with the master branch, all repro tests show no change in results, see:

1 and 2

There is a bit of stuffing around to make this happen:

  • refined CMake to work with esm1.6
  • some constants are configured in the namelist (this is consistent now between esm1.6 and om2, no defaults are changed).
  • for esm1.6 specific code, this is marked as #ifdef ACCESS. #ifdef AusCOM is used for bother esm1.6 and om2, so sometimes #if defined(AusCOM) && !defined(ACCESS) is used for OM2 only. This is used a lot for calendar code.

I snuck in changing the history output precision of the time coordinate to double, technically it's unrelated to this change.

the drivers/access folder comes from the esm1.6 branch. There are minor changes to the drivers/auscom folder to support some constants becoming configurable and improvements made in esm1.6.

@anton-seaice anton-seaice marked this pull request as ready for review January 9, 2026 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants