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

Last Gridcell/Sub routing question #36

Open
mee067 opened this issue Feb 5, 2024 · 3 comments
Open

Last Gridcell/Sub routing question #36

mee067 opened this issue Feb 5, 2024 · 3 comments
Assignees
Labels
documentation Suggested or areas for improvements to documentation question Further information is requested by the issuer

Comments

@mee067
Copy link

mee067 commented Feb 5, 2024

This is just a question. MESH used not to run the LSS for the very last gridcell that has NEXT=0. Newer code, not sure since which release, started to run the LSS for that last gridcell. To make things equivalent to old code, one would set all GRU fractions for that cell to zero for the LSS not to run. Maybe this is related to vector setups as the workflow does not add that additional sub-basin which will have a Rank with NEXT=0 as GK used to do.

I know that last gridcell was added to enable adding its runoff and routing it to the outlet with all the incoming flow.

Now the LSS will generate runoff for that last grid or sub which is not an extra. Does the routing still work for it while its NEXT = 0?

@dprincz dprincz added documentation Suggested or areas for improvements to documentation question Further information is requested by the issuer labels Feb 5, 2024
@dprincz dprincz self-assigned this Feb 5, 2024
@dprincz
Copy link
Contributor

dprincz commented Feb 5, 2024

The routing will only run over 1:NAA, but it will put the outflow from NAA into NA. I don't believe runoff over NA is routed (meaning qo only includes qi from upstream cells). The way to do that is to set NAA=NA. It'll print a warning if that's the case, but I don't believe it will halt the run (I think setups ported from GEM-Hydro usually have the condition NAA=NA).

This is a good consideration, as runoff over the downstream-most subbasin in vector setups might not see full routing (e.g., incorporating runoff over that subbasin itself). It could be something for someone to investigate.

@mee067
Copy link
Author

mee067 commented Feb 6, 2024

From what I gather, this setting of NAA=NA would be inside the code, right?

For large basins that have one outlet at the end, runoff from the last grid/sub may not add much but for smaller basins, and for a setup with multiple outlets, excluding runoff of the last grid/sub or not routing it may be significant.

@dprincz
Copy link
Contributor

dprincz commented Feb 6, 2024

I'm not sure NAA=NA should be an assumption set by the code. It could be set by the fields in the drainage database (e.g., by having no cell with NEXT==0, which could be handled by the workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Suggested or areas for improvements to documentation question Further information is requested by the issuer
Projects
None yet
Development

No branches or pull requests

2 participants