Skip to content
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

Main ifs thermo #612

Merged
merged 11 commits into from
Aug 20, 2024
Merged

Main ifs thermo #612

merged 11 commits into from
Aug 20, 2024

Conversation

dsidoren
Copy link
Collaborator

@dsidoren dsidoren commented Jul 10, 2024

change in the thermodynamics proposed by Dmitry Sein which is required for more physical coupling with IFS type models. In the sea ice step we also neglect the divergent part since its contribution is negligible and sometimes caused errors on levante. The results might be not reproducible!

@dsidoren dsidoren requested review from hegish and JanStreffing and removed request for hegish July 10, 2024 12:00
@JanStreffing JanStreffing added this to the FESOM 2.6 milestone Jul 10, 2024
@JanStreffing JanStreffing self-assigned this Jul 25, 2024
@@ -248,13 +248,13 @@ subroutine ice_timestep(step, ice, partit, mesh)
#endif
#endif /* (__oifs) */
if (flag_debug .and. mype==0) print *, achar(27)//'[36m'//' --> call ice_TG_rhs_div...'//achar(27)//'[0m'
call ice_TG_rhs_div (ice, partit, mesh)
call ice_TG_rhs (ice, partit, mesh)
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 line will lead to the change of the test results


if (flag_debug .and. mype==0) print *, achar(27)//'[36m'//' --> call ice_fct_solve...'//achar(27)//'[0m'
call ice_fct_solve (ice, partit, mesh)

if (flag_debug .and. mype==0) print *, achar(27)//'[36m'//' --> call ice_update_for_div...'//achar(27)//'[0m'
call ice_update_for_div(ice, partit, mesh)
! call ice_update_for_div(ice, partit, mesh)
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 line will lead to the change of the test results

@JanStreffing
Copy link
Collaborator

Here is the tripyview evaluation of a 1958 to 2017 standalone core2 run with these changes vs the same period with the current main branch. I have nowhere to host the html, so as a pdf instead:
fesom26rc_ifs_thermo.pdf

@JanStreffing
Copy link
Collaborator

Patterns look to be similar in magnitude and shape but inverted compared to #86

@JanStreffing
Copy link
Collaborator

Besides the standalone evalulation in the previews comment I also did 50 years AWI-CM3 TCo95L91-CORE2 with constant year 2000 forcing. The results are good; cmpi-tool score is low:
main_ice_thermo
Comparison to before shows only minimal changes:
TCO95L91

I did not output the right MLD (MLD3). Deep ocean salt and temp make no sense to compare after 50 years.

@JanStreffing JanStreffing merged commit 4ee8400 into main Aug 20, 2024
4 checks passed
@JanStreffing JanStreffing deleted the main_ifs_thermo branch August 20, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants