-
Notifications
You must be signed in to change notification settings - Fork 68
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
incremental Mannings n creation with cell-averaging #243
Labels
bug
Something isn't working
Comments
Here you go Keith. Please compare with yours. I think this one is not the final working version.
|
Thanks! we'd also like to support custom default values for this calculation. |
FYI if you have overlapping tiles, the above approach cannot be used. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
When creating the nodal attribute Mannings n landcover parameter, we often have to partition the domain into chunks/slices to fit into available RAM. When this happens, sometimes N>1 cell-averaging stencil overlaps partitions and values can become duplicated leading to errors in the fort.13 file creation for a model such as ADCIRC.
Onur could you please paste here the latest mannings n function we were working on so no one forgets for the next time?
The text was updated successfully, but these errors were encountered: