Skip to content

Notes on CICE5 in ESM1.6#426

Merged
anton-seaice merged 6 commits intomainfrom
80-main
Feb 25, 2026
Merged

Notes on CICE5 in ESM1.6#426
anton-seaice merged 6 commits intomainfrom
80-main

Conversation

@anton-seaice
Copy link
Copy Markdown
Collaborator

Some notes related to #80 . This is a very short summary of the CICE5 configuration for esm1.6, the main reference will be the not-yet-written paper

@ClareCR @blimlim - should we have a seperate page for coupling, and put the meltwater stuff there?

@anton-seaice anton-seaice marked this pull request as ready for review February 4, 2026 02:33
@blimlim
Copy link
Copy Markdown
Collaborator

blimlim commented Feb 4, 2026

@ClareCR @blimlim - should we have a seperate page for coupling, and put the meltwater stuff there?

I think that could be good, though we could merge this as is and then move the meltwater information in a seperate PR for setting up the coupling page?

@anton-seaice
Copy link
Copy Markdown
Collaborator Author

@ClareCR @blimlim - should we have a seperate page for coupling, and put the meltwater stuff there?

I think that could be good, though we could merge this as is and then move the meltwater information in a seperate PR for setting up the coupling page?

I think i will leave it, the iceberg code is within the cice code, and the coupler page could be more about oasis3-mct

Copy link
Copy Markdown
Member

@ccarouge ccarouge left a comment

Choose a reason for hiding this comment

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

Looks good. I spotted a few typos, nothing major. I would keep the meltwater portion and decide later if it needs to move depending on what documentation gets written.

I would approve but fixing the typos is going to cancel my approval anyway, I believe.

Comment thread documentation/docs/pages/model_components/cice.md Outdated
@ccarouge
Copy link
Copy Markdown
Member

ccarouge commented Feb 9, 2026

For next time, my GitHub handle is @ccarouge , "ClareCR" is the handle of Clare, our previous data manager (avoiding giving full name here).

@anton-seaice
Copy link
Copy Markdown
Collaborator Author

For next time, my GitHub handle is @ccarouge , "ClareCR" is the handle of Clare, our previous data manager (avoiding giving full name here).

oops! - I do this all the time

Co-authored-by: Claire Carouge <ccarouge@users.noreply.github.com>
Copy link
Copy Markdown
Collaborator

@blimlim blimlim left a comment

Choose a reason for hiding this comment

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

Thanks @anton-seaice, just a couple small questions

Comment thread documentation/docs/pages/model_components/cice.md Outdated
Comment thread documentation/docs/pages/model_components/cice.md Outdated
Comment thread documentation/docs/pages/model_components/cice.md Outdated
Copy link
Copy Markdown
Collaborator

@blimlim blimlim left a comment

Choose a reason for hiding this comment

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

Thanks @anton-seaice!

Copy link
Copy Markdown
Member

@ccarouge ccarouge left a comment

Choose a reason for hiding this comment

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

LGTM

@anton-seaice anton-seaice merged commit 66ff9a3 into main Feb 25, 2026
24 checks passed
@anton-seaice anton-seaice deleted the 80-main branch February 25, 2026 22:58
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