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

[8pt] GMS catchment being dropped #1280

Open
CarsonPruitt-NOAA opened this issue Sep 10, 2024 · 5 comments · May be fixed by #1284
Open

[8pt] GMS catchment being dropped #1280

CarsonPruitt-NOAA opened this issue Sep 10, 2024 · 5 comments · May be fixed by #1284
Assignees
Labels
bug Something isn't working

Comments

@CarsonPruitt-NOAA
Copy link
Collaborator

Looking at the new HAND v4.5.2.11 in the staging server I've seen a couple of locations where it appears that GMS is not running correctly. The screenshot below is from the Sabin River @ Orange, TX (HUC 12010005). HAND v 4.4.0.0 is on the left and v4.5.2.11 is on the right.
Image

Upon inspection of the v4.5.2.11 data, branch 1477000010 is missing some catchments. I suspect they are being dropped because of the new crosswalking algorithm that drops catchments if it can't snap to a streamline within 100m.
Image

The places I've seen where this is happening seems to be isolated to areas near the coast and seems to be fairly uncommon.

@CarsonPruitt-NOAA CarsonPruitt-NOAA added the bug Something isn't working label Sep 10, 2024
@mluck
Copy link
Contributor

mluck commented Sep 10, 2024

There are a couple of things going on here.

  1. This HUC's outlet levelpaths (colored lines) drain into a lake (shaded blue) in the downstream HUC. Although the original NWM streams (blue lines) do extend past the DEM into the lake, because we trim levelpaths in lakes, the outlet levelpaths don't reach the edge of the DEM (shaded green) and cause reverse-flow issues upstream. The dashed black lines are DEM-derived reaches for branch 0 and the direction of flow is from the east along the yellow levelpath and out through the orange levelpath to the north.
    image

  2. The interaction between reverse flow and pit filling as well as the lack of spatial cohesion between the NWM streams and the channel in the DEM is causing deviations in DEM-derived reaches. Here, the branch 0 reaches (black dashed line) follows the burned in NWM levelpath (purple line). However, the branch 1477000010 line (solid red) more closely follows the depression in the DEM and as suggested in the original issue post above, results in the reach being over the 100m distance used to snap the crosswalk from the NWM stream and so that reach and its catchment are dropped.
    image

@mluck
Copy link
Contributor

mluck commented Sep 11, 2024

I created a patch to remedy the issue above that doesn't clip levelpaths in waterbodies outside of the HUC. But during testing found another issue related to the NWM streams input data, that there is more than one "outlet". There is a canal (red line) that exits the HUC to the east, and since it's an outlet it isn't clipped. Both are being used by branch 0 as the outlets of the HUC.
image

@mluck
Copy link
Contributor

mluck commented Sep 11, 2024

The patched DEM-derived reaches (blue line) fills in the branch 1477000010 reaches that were previously missing (the sections between the red dotted lines), including the original issue (A) and the downstream portion (B).
image

@mluck
Copy link
Contributor

mluck commented Sep 12, 2024

Here's the difference in catchments for branch 1477000010 (v4.5.2.11 catchments in brown, additional new catchments in purple). The yellow line is the old (v4.5.2.11) levelpath and the dashed red line is the new levelpath; NWM levelpaths are in blue.
image

@mluck
Copy link
Contributor

mluck commented Sep 12, 2024

This is the difference in inundation with a 10 year recurrence flow.

v4.5.2.11
image

new
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants