-
Notifications
You must be signed in to change notification settings - Fork 0
Master esm1.6 merge #68
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
base: master
Are you sure you want to change the base?
Conversation
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.
CICE actually uses a proleptic gregorian calendar, see https://cfconventions.org/Data/cf-conventions/cf-conventions-1.12/cf-conventions.html#calendar
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>
a8c152b to
29a9581
Compare
|
Compare of this morning's work: |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file follows the access-esm1.6 branch except for minor formatting changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/ACCESS-NRI/cice5/compare/access-esm1.6..ACCESS-NRI:cice5:master-esm1.6-merge shows no changes to this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/ACCESS-NRI/cice5/compare/access-esm1.6..ACCESS-NRI:cice5:master-esm1.6-merge shows no changes to this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/ACCESS-NRI/cice5/compare/access-esm1.6..ACCESS-NRI:cice5:master-esm1.6-merge shows no changes to this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/ACCESS-NRI/cice5/compare/access-esm1.6..ACCESS-NRI:cice5:master-esm1.6-merge shows no change to this file
29a9581 to
adef59d
Compare
Read grid on each processor per COSIMA/access-om2#221
fix line truncation build failure
adef59d to
99ce18a
Compare
|
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: There is a bit of stuffing around to make this happen:
I snuck in changing the history output precision of the time coordinate to double, technically it's unrelated to this change. the |
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