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

multi-pass mesh generation using a fort.64 seed #69

Open
HamishB opened this issue Apr 17, 2020 · 2 comments
Open

multi-pass mesh generation using a fort.64 seed #69

HamishB opened this issue Apr 17, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@HamishB
Copy link

HamishB commented Apr 17, 2020

Hi, an idea for your consideration--

I'm wondering if it could be useful to do multi-pass mesh generation by:

  • using the existing M2 wavelength mesh size function with your input DEM in the usual manner,
  • run ADCIRC with the resulting mesh and output a fort.64 file (Depth-averaged Velocity Time Series at All Nodes in the Model Grid),
  • then read that fort.64.nc file back into OM2D,
  • and finally run a new mesh size function based on observed velocities.

The idea being to better focus the mesh effort into areas of known high flow that the M2 shallow water wave eq. might not specifically capture, e.g. tidal straits or the inlet of an estuary which includes a large river flow as a boundary condition.

i.e. use a 'real' value for |u| in the CFL calculation instead of just estimating it.

At minimum it could be used to simply plot a map view highlighting areas where your mesh is either too coarse or needlessly fine, based on initial fort.64 results.

@krober10nd
Copy link
Collaborator

Hey Hamish,

I don't think it's a bad idea. We've considered this in the past, but we've always stuck to a more a priori workflow based on linear waves. What I think would make more sense is to use the max vel file to get the modeled |u| for the Courant number calculation. That really isn't difficult as it just requires msh.BoundCr takes a varargin with the modeled velocity.

Then you could pass the point set of the mesh you just simulated with and a new sizing function based on the updates to the Courant number and run the generation for a couple iterations.

@HamishB
Copy link
Author

HamishB commented Apr 20, 2020

What I think would make more sense is to use the max vel file to get the modeled |u| for the Courant number calculation.

Yes, that would be much simpler.

@krober10nd krober10nd added the enhancement New feature or request label Apr 27, 2020
@krober10nd krober10nd self-assigned this Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants