-
Notifications
You must be signed in to change notification settings - Fork 73
Correct temperature and salinity mean diagnostics when using conservative/absolute #860
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
Correct temperature and salinity mean diagnostics when using conservative/absolute #860
Conversation
I believe the checks that failed are expected. My inclination is to not wrap "bugfix" changes around these unless necessary since it is only diagnostics that change. But I'm unsure if it will impact other users, I'm not sure if anyone else has started using the TEOS-10 EOS. |
It looks to me like they are identical but in a different order? |
Oh good point, yes in these runs they should be identical. They will only change for cases that used the conservative temp/absolute salinity. |
- Previously the global/area mean temperature and salinity outputs were the model prognostic temp/salt, but were always reported as potential temperature and practical salinity. - This gives the correction for those outputs so it is actually potential temperature and practical salinity, even when the model is conservative and absolute. - The option is also added to output the averaged conservative temperature and absolute salinity variables when those are the prognostic variables. - The option to output conservative temp and absolute salinity when model is potential temp and practical salinity is not added.
5e4beb6
to
a11329d
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev/gfdl #860 +/- ##
============================================
- Coverage 38.07% 38.06% -0.01%
============================================
Files 298 298
Lines 87946 87988 +42
Branches 16508 16520 +12
============================================
+ Hits 33482 33490 +8
- Misses 48411 48446 +35
+ Partials 6053 6052 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
We decided to let this in for now and then (hopefully) try to update the testing to handle this before it gets shipped to mom-ocean.
Gaea regression: https://gitlab.gfdl.noaa.gov/ogrp/mom6ci/MOM6/-/jobs/151913 ✔️ 🟡 New diagnostics added to |
This addresses issue #858.