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

Shallow Water #70

Merged
merged 27 commits into from
Nov 18, 2024
Merged

Shallow Water #70

merged 27 commits into from
Nov 18, 2024

Conversation

garrettbyrd
Copy link
Collaborator

This PR adds the linear shallow water model in 2d with some examples included.

Documentation has been updated to:

  • define the linear shallow water 2D model in the Models section
  • describe a tutorial for a gaussian "drop" in a square domain with no-normal-flow boundary conditions.

The linear shallow water model does not include GPU acceleration (similar implementation to Burgers1D). No-normal-flow boundary conditions are implemented in the base model.

@garrettbyrd garrettbyrd marked this pull request as ready for review November 12, 2024 20:23
@fluidnumerics-joe
Copy link
Member

To fix fprettify-lint failure, run the following

fprettify  './src/' --config-file ./fprettify.config --recursive --case 1 1 1 1
fprettify  './test/' --config-file ./fprettify.config --recursive --case 1 1 1 1
fprettify  './examples/' --config-file ./fprettify.config --recursive --case 1 1 1 1

And commit & push the changes up to the repo.. I'm sure we could write a bot for this at some point...

@fluidnumerics-joe
Copy link
Member

I would increase the time limit for the github workflows -- the debug and coverage builds run quite slowly.

@fluidnumerics-joe
Copy link
Member

/superci

Copy link

codecov bot commented Nov 14, 2024

Codecov Report

Attention: Patch coverage is 95.16129% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/SELF_LinearShallowWater2D_t.f90 97.36% 1 Missing ⚠️
src/cpu/SELF_LinearShallowWater2D.f90 0.00% 1 Missing ⚠️
src/gpu/SELF_LinearShallowWater2D.f90 95.65% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@fluidnumerics-joe fluidnumerics-joe self-requested a review November 14, 2024 19:55
@fluidnumerics-joe
Copy link
Member

In addition to the above inline review comments, you'll need to add the Linear Shallow Water (2D) model and your tutorial to the appropriate spot in the nav: in the mkdocs.yml; currently, this model doesn't appear in the navigation on the rendered site. Other than those things, this is looking good.

@fluidnumerics-joe
Copy link
Member

@garrettbyrd - We need to get full gpu offloading implemented for this one as well

examples/CMakeLists.txt Outdated Show resolved Hide resolved
docs/Models/linear-shallow-water-model.md Outdated Show resolved Hide resolved
docs/Models/linear-shallow-water-model.md Outdated Show resolved Hide resolved
docs/Models/linear-shallow-water-model.md Outdated Show resolved Hide resolved
@fluidnumerics-joe fluidnumerics-joe merged commit 54d5201 into main Nov 18, 2024
11 checks passed
@fluidnumerics-joe fluidnumerics-joe deleted the model/shallow-water-2d branch November 18, 2024 15:07
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.

2 participants