-
Notifications
You must be signed in to change notification settings - Fork 86
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
Underwater terrain-aware wave forcing #1475
Underwater terrain-aware wave forcing #1475
Conversation
…ag forcing) happens
Once this gets a regression test, we can upgrade it to "ready for review". |
Main point of PR: allow terrain drag to do its thing underwater. Wave forcing modifies velocity fields within the relaxation zones -- now it will leave cells untouched that are "blanked" (inside terrain) or marked as drag cells (velocity is modified by drag term). Wave boundary fill sets the boundary velocity to the target velocity of the wave profile -- now it will set the velocity to 0 in boundary cells that are adjacent to "blanked" cells so no flow is pushing directly into terrain at the boundary. |
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.
Cool work!
We should consider adding a walk through using Will's case. This PR adds many different capabilities in a single solve. |
That's a good idea! I'm getting the capability in for the version release, but I think there will be more development and documentation as Matt and Will continue to look into this. |
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.
Cool work! Have just a super minor comment
Summary
Turn off wave forcing where there is terrain, and modify ocean wave boundary to also be aware.
Pull request type
Please check the type of change introduced:
Checklist
The following is included:
This PR was tested by running:
Additional background
For continuing collaboration with water power