forked from E3SM-Project/E3SM
-
Notifications
You must be signed in to change notification settings - Fork 7
Add wind forcing and bottom drag tendency terms #219
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
Merged
mark-petersen
merged 31 commits into
E3SM-Project:develop
from
mwarusz:omega/wind-forcing-and-bottom-drag
Jun 12, 2025
Merged
Changes from 29 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
768efd1
Add wind forcing aux vars
mwarusz 2c2e681
Refactor test helpers to also handle 1D fields
mwarusz e388683
Add a test for WindForcingAux
mwarusz 2032459
Add wind forcing tendency term
mwarusz 7e59f74
Test wind forcing tendency term
mwarusz 2bd5ae0
Fix single precision tend test not running on planar mesh
mwarusz 3e5bc43
Add bottom drag tendency term
mwarusz b20d41b
Add unit test for bottom drag tendency term
mwarusz 7903896
Add computation of normal wind to wind forcing aux
mwarusz b467a8f
Add option to use absolute tolerance in test helpers
mwarusz c8f0607
Reformulate wind forcing in terms of stress
mwarusz 28b1a0a
Add wind forcing and bottom drag to Tendencies
mwarusz da040da
Remove "Default" in aux vars names
mwarusz d75b2e6
Rename wind stress input fields
mwarusz e0ba780
Update docs
mwarusz b502913
Add exchangeHalo function to AuxiliaryState
mwarusz 602c6b1
Exchange auxiliary state halo in ocnInit
mwarusz fa7e3e5
Fix names of wind forcing aux vars in docs
mwarusz b4ed972
Use density from config in wind forcing tendency
mwarusz 3b549a0
Add InterpCellToEdge horizontal operator
mwarusz 677d981
Use InterpCellToEdge in WindForcingAux
mwarusz 1d3c340
Init Pacer in time stepper tests
mwarusz 3b2cced
Remove halo exchange of wind stress at every step
mwarusz e98791a
Compute wind forcing aux in 1d parallelFor
mwarusz 2bd6c87
Fix time stepper test
mwarusz f5599f3
Compute bottom drag in 1d parallelFor
mwarusz 97139ab
Merge branch 'develop' into omega/wind-forcing-and-bottom-drag
philipwjones 45f937c
fix bugs and linting errors introduced during WindForcing merge
philipwjones dfbd2c6
add readConfigOptions for testAuxState
mark-petersen 388b3cc
add default for Enabled in BottomDragOnEdge constructor
mark-petersen 4c1caf6
fix spacing
mark-petersen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
generic comment - do we want this in AuxState? Would it make sense to have a 'forcing module' where this lives?
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.
These are grouped in the
WindForcingAuxVarsclass, so they aren't directly members of the aux state class. This is mentioned in the dev docs, but maybe it should also included be in the user docs.