-
Notifications
You must be signed in to change notification settings - Fork 33
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
[1pt] PR: BARC updates to cap xsec area and allow user to choose input bankfull geometry #393
Conversation
- bathy_xs_area_chg_flag set to 1.0 - replace flagged xs_area with BANKFUL_XSEC_AREA
- Improves dislapy of having FIM rating on top of USGS
- Aggregate all of the individual huc elev_tables into one aggregate for accessing all data in one csv
- Facilitates user input to change the bankfull dataset used in BARC
- Can use these environ variables to switch the input bankfull geom - Default is to use the Wieczorkek et al. (2018) data (Bieger 2015 regression eq)
@RyanSpies-NOAA Please update the Changelog for this branch |
Change log draft message: v3.0.17.0 - 2021-06-04 - PR #393BARC updates to cap the bathy calculated xsec area in Changes
|
@RyanSpies-NOAA Do you recommend I run a test on the VM to confirm successful runs by a different user? If so, please paste the desired usage for me to try. If not, I will go ahead and approve as this feature can be activated or deactivated easily, which is great. |
@BradfordBates-NOAA probably not essential but feel free to run a quick test with the following (random HUC over Omaha, NE): |
These revisions aim to improve the technique and structure used for the bathymetry estimation. There are also some minor enhancements to the rating curve comparison workflow. This feature branch addresses the following github tickets:
Changes
params_calibrated.env & params_template.env
bathy_rc_adjust.py
run_by_unit.sh
rating_curve_comparison.py
Screenshots
Example of a new rating curve comparison plot with the FIM synthetic rating curve now plotted on top of USGS rating curve:

Testing the NWM route link bankfull geometry
This feature branch was also used to evaluate the FIM Bathymetry Adjusted Rating Curve (BARC) performance using the estimated bankfull geometry dataset derived for the NWM route link dataset (#343). The goal of this evaluation was to determine if one input bankfull geometry dataset clearly performed better than the other. Results were analyzed for three different FIM versions:
FIM 3_0_16_1: the most recent FIM dev version at the time of evaluation
FIM NWMGeom: the input channel geometry database was developed using the CONUS derived regression equations from Blackburn-Lynch et al. (2017).
FIM Bieger: the input channel bankfull geometry dataset was developed by Wieczorek et al. (2018) using the Bieger et al. 2015 regional regression equations. This FIM version is the same as FIM 3_0_16_1 except that BARC now includes a bathymetry cross section area calculation limit to prevent values from exceeding the bankfull cross section regression value (see changes above).
Alpha Evaluation Summary
The area based CSI metrics were used to compare the mainstem FIM performance between FIM 3_0_16_1, FIM NWMGeom, and FIM Bieger. Aggregated area based CSI scores indicate that inundation outputs (with BARC) using the NWMGeom produce a slight regression for the action and minor thresholds but also produce a small improvement for the moderate and major thresholds:


Site specific results (median CSI score for all sites) hint at more noticeable regression for action and minor thresholds for the FIM NWMGeom compared to the FIM Bieger:


Sierra Evaluation Summary
The rating curve comparison evaluation compares the FIM synthetic rating curve vs. USGS rating curve for the same FIM versions outlined in the alpha evaluation above. Looking at the mean absolute elevation difference (units=feet) for a range of flow thresholds (1.5yr, 5yr, 10yr, action, minor, moderate, major) can provide and indication of synthetic rating curve performance between the different FIM simulations.
The following plot shows the histogram of "Mean Absolute Elev Differences" at 361 USGS gage sites. The Mean Absolute Elev Difference was calculated for the two FIM versions (FIM NWMGeom and FIM Bieger) and the resulting delta (FIM Bieger minus FIM NWMGeom) indicate which FIM version performed better. Negative bins (orange) in the plot below show the number of instances where FIM Bieger performed better than FIM NWMGeom and the positive bins (blue) indicate the inverse.

Conclusions
Based on these findings, the FIM Dev team decided to continue using the FIM Bieger version as the default input in the FIM workflow. Further development in the BARC methodology will likely evolve over time, and the FIM performance using NWMGeom (or other version) could potentially exceed the current FIM Bieger performance. The capability now exists to test the different channel bankfull geometry datasets seamlessly.