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] Update CatFIM to accommodate sites where the flood levels are elevations, not stages #1384

Open
EmilyDeardorff opened this issue Dec 19, 2024 · 0 comments
Labels
bug Something isn't working CatFIM NWS Flood Categorical HAND FIM

Comments

@EmilyDeardorff
Copy link
Contributor

Based on vlab ticket #141569.

At site prdk2 in Paradise, KY, the observations, forecasts, and flood thresholds are elevations rather than stage values. This means that stage-based CatFIM is inundating that site up to 378 ft for 'Action' stage, which is creating an inundated area over 10 miles wide.

Write a workaround into the stage-based CatFIM code to detect and mitigate this issue. Then test that this workaround is only mitigating sites with this issue (and not areas where the zero datum is just a very small number).

The workaround should look something like this:

if action_stage > zero_datum:
action_stage =- zero_datum

The following sites will likely also have this issue: CENI3, CLFI3, PNXP1, PVLK2, and ISVO1.

Current behavior

image

Expected behavior

A much smaller inundated area is expected for action stage at this site.

@EmilyDeardorff EmilyDeardorff added bug Something isn't working CatFIM NWS Flood Categorical HAND FIM labels Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CatFIM NWS Flood Categorical HAND FIM
Projects
None yet
Development

No branches or pull requests

1 participant