Consider surface ocean current in air-sea flux calculation #755
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…and allow one to override data integrity checks in FMS restart logic
…hafsv0.3_final Conflicts: atmos_cubed_sphere
…nLiu-NOAA): suite_FV3_HAFS_v0_thompson.xml suite_FV3_HAFS_v0_thompson_nonsst.xml suite_FV3_HAFS_v0_thompson_noahmp.xml suite_FV3_HAFS_v0_thompson_noahmp_nonsst.xml
…sing the prime meridian.
…ling time steps). This is to reduce the overhead introduced by downscaling the coupling variables from FV3ATM parent to nest.
…branch of the hafs-community fork.
…unity/fv3atm into feature/hafsv0.3_final
…FS sa-TKE EDMF PBL scheme for HAFS/hurricane modeling.
in internal tracker output (fort.602, phtcf file).
…re and ccpp-physics.
…#622) * fix initialization for moving nest grid
41 tasks
…hafsv2_baseline_with_ssc
and suite_FV3_HAFS_v1_thompson_nonsst.xml under ccpp/suites since they are used by operational HAFS configurations.
DeniseWorthen
approved these changes
Jan 31, 2024
BinLiu-NOAA
approved these changes
Jan 31, 2024
junwang-noaa
approved these changes
Jan 31, 2024
|
ufs-community/ccpp-physics#155 was merged. This PR branch is ready to revert .gitmodules and commit the latest ufs/dev hash of ccpp-physics: ufs-community/ccpp-physics@51452b8 |
Thanks, @grantfirl! Waiting for NOAA-GFDL/GFDL_atmos_cubed_sphere/pull/311 being merged. |
|
atmos_cubed_sphere hash: NOAA-GFDL/GFDL_atmos_cubed_sphere@6663459 |
|
@jkbk2004 We have approvals here, can you please merge this PR? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR will update ccpp-physics and GFDL_atmos_cubed_sphere components to include surface ocean current in the computation of air-sea fluxes.
This PR will also add a new namelist called "icplocn2atm."
When icplocn2atm=0, surface ocean current is not used in the computation of air-sea fluxes.
When icpocn2atm=1, surface ocean current is used in the computation of air-sea fluxes.
Issue(s) addressed
Testing
How were these changes tested? The changes were tested in the HAFS-MOM6 system.
What compilers / HPCs was it tested with? Intel
Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
New tests will not be added to UFS regression test system.
Have the ufs-weather-model regression test been run? On what platform? Yes, on Hera.
Dependencies
If testing this branch requires non-default branches in other repositories, list them.
Those branches should have matching names (ideally)
Do PRs in upstream repositories need to be merged first?
If so add the "waiting for other repos" label and list the upstream PRs
Co-authored-by: @binli2337, @BinLiu-NOAA, @WeiguoWang-NOAA, @BijuThomas-NOAA, @hyunsookkim-NOAA, @XuLi-NOAA, @MariaAristizabal-NOAA, @JohnSteffen-NOAA