Skip to content

Expose ksno (thermal conductivity of snow ) to the namelist#41

Merged
anton-seaice merged 2 commits intoaccess-esm1.6from
ksno
Jun 16, 2025
Merged

Expose ksno (thermal conductivity of snow ) to the namelist#41
anton-seaice merged 2 commits intoaccess-esm1.6from
ksno

Conversation

@anton-seaice
Copy link
Copy Markdown
Collaborator

Typical default value is 0.3, for CM2 - use 0.2

This is just for future compatibility with CM2

(related to #32 )

Typical default value is 0.3, for CM2, use 0.2
@anton-seaice anton-seaice requested a review from dougiesquire June 4, 2025 23:57
@anton-seaice anton-seaice self-assigned this Jun 4, 2025
Copy link
Copy Markdown

@dougiesquire dougiesquire left a comment

Choose a reason for hiding this comment

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

Typical default value is 0.3, for CM2 - use 0.2

Just to clarify. The default value is now 0.3 and 0.2 will be set for CM2 via the namelist?

Comment thread source/ice_init.F90 Outdated
ice_ref_salinity = 5._dbl_kind ! reference salinity for ice–ocean exchanges (ppt)
ice_ref_salinity = 5._dbl_kind, & ! reference salinity for ice–ocean exchanges (ppt)
! n.b. CICE6 uses 4 ppt
ksno = 0.3_dbl_kind ! thermal conductivity of snow (W/m/deg)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Are you sure you want to change the value used when #ifndef AusCOM?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think so? 0.3 is the default in cice4 and the master branch etc

Co-authored-by: Dougie Squire <42455466+dougiesquire@users.noreply.github.com>
@anton-seaice anton-seaice merged commit 267d4be into access-esm1.6 Jun 16, 2025
4 checks passed
@anton-seaice anton-seaice deleted the ksno branch June 16, 2025 01:08
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.

2 participants