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

[5pt] Create and implement a restricted sites list to mitigate overlapping sites in flow-based CatFIM #1396

Open
EmilyDeardorff opened this issue Jan 7, 2025 · 1 comment
Labels
CatFIM NWS Flood Categorical HAND FIM enhancement New feature or request

Comments

@EmilyDeardorff
Copy link
Contributor

EmilyDeardorff commented Jan 7, 2025

Currently we have a restricted sites list for stage-based CatFIM but we do not have one for flow-based CatFIM. Create a restricted sites list with the same structure as the stage-based restricted sites list.

The first site to add is CMUG1, which has been flagged in vlab ticket #141798 as being mistakenly included (since it has been replaced by a different point).

Analyze the existing CatFIM libraries to look for any other CatFIM sites where there's incorrect CatFIM from overlapping sites and mitigate using the restricted sites list.

@EmilyDeardorff EmilyDeardorff added enhancement New feature or request CatFIM NWS Flood Categorical HAND FIM labels Jan 7, 2025
@RobHanna-NOAA
Copy link
Contributor

Oh.. good point. Do we want a seperate one for FLOW? or maybe a more advance CSV with true/false colums for "for Stage Based", "For Flow based". etc. Not sure the right answer here but you an figure it out. But one csv might make it easier for Derek to take to the field for edits. And we should bug Derek about getting updates for this before the next full CatFIM build release. :)

@EmilyDeardorff EmilyDeardorff changed the title [3pt] Create and implement a restricted sites list for flow-based CatFIM [5pt] Create and implement a restricted sites list to mitigate overlapping sites in flow-based CatFIM Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CatFIM NWS Flood Categorical HAND FIM enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants