NOAA-OWP / t-route

Tree based hydrologic and hydraulic routing
Other
44 stars 50 forks source link

Fixed broken diffusive routing in Lower Colorado example #805

Closed JurgenZach-NOAA closed 4 months ago

JurgenZach-NOAA commented 4 months ago

Recent updates caused the diffusive routing for NHD in the Lower Colorado example to no longer worked.

Additions

Removals

Changes

Testing

  1. In the test/LowerColorado_TX directory, issue: >> python -m nwm_routing -f -V3 test_AnA.yaml

Screenshots

Notes

-

Todos

-

Checklist

Testing checklist

Target Environment support

Accessibility

Other

AminTorabi-NOAA commented 4 months ago

@JurgenZach-NOAA If you uncomment output_parameters: and stream_output to save the results. It will throw an error that AttributeError: 'str' object has no attribute 'keys'. Can you fix the error

JurgenZach-NOAA commented 4 months ago

@JurgenZach-NOAA If you uncomment output_parameters: and stream_output to save the results. It will throw an error that AttributeError: 'str' object has no attribute 'keys'. Can you fix the error

This issue was not actually related to the PR - however, it's corrected now. What happened was that, in cases without assimilated gages and NHD networks, link_lake_crosswalk was set to "None" instead of (Boolean) False. So, the output routine thought there's a link_lake_crosswalk when there couldn't be. Corrected it.

JurgenZach-NOAA commented 4 months ago

Also turned on the output in the example - there should always be an output!

JurgenZach-NOAA commented 4 months ago

image

nwm_out_generator in the V3 main branch had a missing argument, the duplicate_ids_df. It is only relevant for HYfeatures, so it can be set to an empty df here, but still needs to be provided.