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

Terrain blanking informed by ocean waves #1369

Draft
wants to merge 34 commits into
base: main
Choose a base branch
from

Conversation

mbkuhn
Copy link
Contributor

@mbkuhn mbkuhn commented Nov 22, 2024

Summary

Leveraging the combination of the IB forcing method for terrain with the wave profiles used in a numerical wave tank setting.

Pull request type

Please check the type of change introduced:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Checklist

The following is included:

  • new unit-test(s)
  • new regression test(s)
  • documentation for new capability

This PR was tested by running:

  • the unit tests
    • on GPU
    • on CPU
  • the regression tests
    • on GPU
    • on CPU

@mbkuhn mbkuhn force-pushed the moving_terrain_blanking branch from 2587b10 to 8b4090a Compare November 22, 2024 22:47
@mbkuhn
Copy link
Contributor Author

mbkuhn commented Nov 25, 2024

Linear Waves as Terrain

Evidence that all of the right pieces are showing up. The blanked cells in blue, the drag cells in red, the ABL flow above the waves, and the wave velocity profile as the target velocity within the blanked cells (arrows).

@mbkuhn
Copy link
Contributor Author

mbkuhn commented Nov 25, 2024

W2A waves as terrain

Working for the waves2amr pathway, i.e. using wave modes from HOS-Ocean. Getting the right velocity inside the waves requires the bug fix #1372 (already included).

@mbkuhn
Copy link
Contributor Author

mbkuhn commented Nov 25, 2024

Will be meeting with Harish to hash out the question marks on the surface + turbulence modeling sides.

@mbkuhn
Copy link
Contributor Author

mbkuhn commented Nov 25, 2024

After discussing with Harish, we concluded that this PR is mostly there. The main thing that remains is adapting the wall-model forcing term (applied to cells flagged as "drag" cells). We may consult Manuel Ayala's work for this.

Other possible concerns are the magnitude of the Cd term (applied to cells flagged as "blanked" cells) - too low may mean that the velocity within the waves doesn't change fast enough, and too high may mean stability issues - and overlap of waves and sponge forcing near outlets - may need to smooth waves at inflow or outflow. But these concerns have to do with robustness rather than accuracy.

Other design choices: for now, we are assuming the ocean surface has a uniform temperature in a single simulation. If the simulations get bigger, we may need to change that. We are also assuming a uniform roughness over the waves (z0). This parameter represents the underresolved waves, not the resolved ones that are modeled as terrain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant