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

Workbench fixsrfstress #120

Merged
merged 8 commits into from
Aug 20, 2024
Merged

Workbench fixsrfstress #120

merged 8 commits into from
Aug 20, 2024

Conversation

patrickscholz
Copy link
Contributor

exchange oce-atmos (stress_atmoce_x(n)) surface stress when computing KPP and MOMIX with the total surface stress (stress_iceoce_x(n)a_ice(n) + stress_atmoce_x(n)(1.0_WP-a_ice(n)))

It has surprisingly less effect after one full JRA55 cycle, but nevertheless the github testcase will of course fail.

image

image

@koldunovn
Copy link
Member

koldunovn commented May 3, 2021

@patrickscholz The question is why ICEPACK tests did not fail? :) @lzampier Does it affects you somehow?

@lzampier
Copy link
Collaborator

lzampier commented May 3, 2021

Sorry I cannot follow the discussion at the moment. I need a bit more context. I am having a look at the code right now.

@lzampier
Copy link
Collaborator

lzampier commented May 3, 2021

It's strange because call oce_fluxes_mom is done in the same way for both versions... Is it possible to see the namelists used for the test?

@patrickscholz
Copy link
Contributor Author

its rather the actual default namelists you can check in /home/ollie/pscholz/fesom2.0_github/work_test_fixsrfstress_ctrl

@JanStreffing JanStreffing added this to the FESOM 2.6 milestone Feb 28, 2024
@JanStreffing JanStreffing changed the base branch from master to refactoring February 28, 2024 12:36
@JanStreffing JanStreffing changed the base branch from refactoring to main June 21, 2024 08:06
@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.
fesom26rc_fixsrfstress.pdf

@patrickscholz, I see some changes, especially in the Tropical upper ocean. Does this look ok to you?

@patrickscholz
Copy link
Contributor Author

@JanStreffing : Feels a bit strange since the main effect of this bug fix should not be in the equatorial area. There, not much should have changed, if there are changes than they should rather happen in the icecovered areas? When i made the testcase a while ago (see header figures) the effect looked way much smaller. Although i plotted the mean over the upper 250m

@JanStreffing
Copy link
Collaborator

I might have made a mistake in my evaluation run. I made the reference run, and all other experiments for branches that change the results at the same time. I made this test simulation here more recently. I think that while updating this branch in preparation to merge, I may have merged some other the other result changing merges into here. In that case the difference, we see, would be the sum of multiple branches.

I will make a new set of experiments with current main and this branch, to be 100% sure.

@JanStreffing
Copy link
Collaborator

All clear. The new test that compares this branch to main has an order of magnitude smaller differences, and they are where we expect them, in high latitude regions:
fesom26rc_fixsrfstress2.pdf

@JanStreffing JanStreffing self-requested a review August 20, 2024 14:27
@JanStreffing JanStreffing merged commit ccee202 into main Aug 20, 2024
4 checks passed
@JanStreffing JanStreffing deleted the workbench_fixsrfstress branch August 20, 2024 14:35
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.

4 participants